Skip to content
This repository has been archived by the owner on Jan 8, 2020. It is now read-only.

Removed impossible condition #6850

Closed
wants to merge 1 commit into from

Conversation

mesmerizero
Copy link

If $this->curl would be true-ish when entering this method it would enter inside the first conditional block and inside $this->close() will be set to null. Then when checking the second condition, that I removed, it will always logically evaluate to false and the block will be never entered regardless of the input or context.

If $this->curl would be true-ish when entering this method it would enter inside the first conditional block and inside $this->close will be set to null. Then when checking the second condition, that I removed, it will always logically evaluate to false.
@Ocramius Ocramius self-assigned this Nov 22, 2014
@Ocramius Ocramius modified the milestones: 2.3.4, 2.4.0 Nov 22, 2014
Ocramius added a commit that referenced this pull request Nov 22, 2014
@Ocramius
Copy link
Member

@mesmerizero merged into develop at 5164482, thank you!

@Ocramius Ocramius closed this Nov 22, 2014
gianarb pushed a commit to zendframework/zend-http that referenced this pull request May 15, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants