Skip to content

Conversation

@paradajozsef
Copy link
Contributor

Q A
Bug fix? no
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets -
License MIT
Doc PR -

This return is unreachable after #17427.

@xabbuh
Copy link
Member

xabbuh commented Feb 27, 2016

👍

Status: Reviewed

@fabpot
Copy link
Member

fabpot commented Feb 28, 2016

Good catch, thanks @paradajozsef.

@fabpot fabpot merged commit 8f2d5bb into symfony:master Feb 28, 2016
fabpot added a commit that referenced this pull request Feb 28, 2016
…zsef)

This PR was merged into the 3.1-dev branch.

Discussion
----------

[Process] Remove unreachable return statement.

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | -
| License       | MIT
| Doc PR        | -

[This return](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Process/Process.php#L1267) is [unreachable](https://github.com/symfony/symfony/blob/master/src/Symfony/Component/Process/Process.php#L1255) after #17427.

Commits
-------

8f2d5bb [Process] Remove unreachable return statement.
@paradajozsef paradajozsef deleted the remove-unreachable-return-statement branch January 13, 2017 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants