Add root namespace character #3340

Closed
wants to merge 2 commits into
from

Conversation

Projects
None yet
2 participants
@iwalz
Contributor

iwalz commented Jan 3, 2013

The backslash for the root namespace of openssl_error_string() is not defined. Should solve #3332

@iwalz

This comment has been minimized.

Show comment
Hide comment
@iwalz

iwalz Jan 3, 2013

Contributor

The first would simply stop to confuse with the error message. I guess it's not expected, that this extension is loaded, it simply clears the openssl error buffer and displays it. If the extension is not loaded, the loop is not executed and the user will get less of information on error (nothing else in this class relies on the openssl_* commands)

Contributor

iwalz commented Jan 3, 2013

The first would simply stop to confuse with the error message. I guess it's not expected, that this extension is loaded, it simply clears the openssl error buffer and displays it. If the extension is not loaded, the loop is not executed and the user will get less of information on error (nothing else in this class relies on the openssl_* commands)

weierophinney added a commit that referenced this pull request Jan 4, 2013

@ghost ghost assigned weierophinney Jan 4, 2013

weierophinney added a commit to zendframework/zend-http that referenced this pull request May 15, 2015

weierophinney added a commit to zendframework/zend-http that referenced this pull request May 15, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment