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

zend_hash_do_resize: fix compacting condition #964

Merged
merged 1 commit into from Dec 23, 2014

Conversation

3 participants
@kaja47
Copy link
Contributor

kaja47 commented Dec 23, 2014

nNumUsed should always be greater or equal to nNumOfElements so original condition is never true and arrays are always doubled in size and compaction is never triggered

zend_hash_do_resize: fix compacting condition
nNumUsed should always be greater or equal to nNumOfElements so original condition is never true and arrays are always doubled in size and compaction is never triggered
@laruence

This comment has been minimized.

Copy link
Member

laruence commented Dec 23, 2014

nice catch, thanks, merged

@php-pulls php-pulls merged commit c462ff7 into php:master Dec 23, 2014

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment