Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix error if compiled without -fpermissive flag. #2588

Closed
wants to merge 1 commit into from

Conversation

ideal
Copy link
Contributor

@ideal ideal commented Jun 21, 2017

No description provided.

@krakjoe
Copy link
Member

krakjoe commented Jun 23, 2017

This should target 7.0, and another patch is required for master because smart string was moved to Zend.

@ideal
Copy link
Contributor Author

ideal commented Jun 23, 2017

@krakjoe So do i need to open another two separate pull requests and cancel this one?

@nikic
Copy link
Member

nikic commented Jun 23, 2017

Merged as 4d61005 into 7.0+. Thanks!

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.

3 participants