Skip to content

Improved zend_string API #1367

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

Closed
wants to merge 2 commits into from
Closed

Conversation

flaupretre
Copy link
Contributor

This PR complements the zend_string API with some missing macros and functions. It also unifies the macro names to use the 'ZSTR_' prefix (compatibility macros for the previous names are provided).

The code using these macros has been modified to use the new 'ZSTR' names. That's cleaner but not mandatory. If you think that it represents too many changes, don't merge the second commit.

This PR contains only additions. It does not create any BC break.

Add missing methods
Does not change anything to the generated code (thanks to compat macros) but cleaner.
@php-pulls
Copy link

Comment on behalf of ab at php.net:

Merged by Dmitry. Thanks for the contribution, Francois.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants