-
Notifications
You must be signed in to change notification settings - Fork 7.9k
ext/zip: Remove un-needed check for zip_open #4189
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
Conversation
@nikic, this is waiting for a review whenever you get time. |
@hughmcmaster As said before, I'm not sure that removing those |
This version removes the un-necessary A future PR will replace Thank you to everyone for your patience and explanations on the issues in the previous PR versions. |
Updated per review feedback and upstream changes.