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

SolrClient::optimize(): zpp segfault fix + parse int macros #3 #53

Merged
merged 16 commits into from
Oct 25, 2023
Merged

SolrClient::optimize(): zpp segfault fix + parse int macros #3 #53

merged 16 commits into from
Oct 25, 2023

Conversation

omars44
Copy link
Collaborator

@omars44 omars44 commented Oct 18, 2023

Fixes optimize segfault in php with debug mode.
optimize: Changed signature
internals: parse int macros (strict and non-strict modes)

Copy link
Collaborator

@bix0r bix0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like the new macros, but I have concerns regarding the signature change.

src/php7/solr_macros.h Outdated Show resolved Hide resolved
src/php7/solr_macros.h Outdated Show resolved Hide resolved
src/php7/php_solr_client.c Show resolved Hide resolved
@omars44
Copy link
Collaborator Author

omars44 commented Oct 21, 2023

@bix0r Addressed all the comments

Copy link
Collaborator

@bix0r bix0r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 👍
Just fix that unused line

src/php7/solr_macros.h Outdated Show resolved Hide resolved
@omars44 omars44 merged commit 053ab59 into php:master Oct 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants