Fixed bug #62202 (ReflectionParameter::getDefaultValue() leaks with constand #97

Closed
wants to merge 1 commit into
from

3 participants

@reeze

Hi,
I found this leaks when testing reflection related functions.

This happend because when compiling constant zval.type will add IS_CONSTANT_UNQUALIFIED.

Thank:)

@laruence
php.net member

why my comment disappeared? what did you do?

@reeze

it lost when I force-updated the PR,
because I want to update the zval_type to type to make it short and clean.
It's still in here reeze@34407d9#commitcomment-1402443

but this commit is not connected with this PR anymore so It dispeared in this conversation :(

@laruence
php.net member

thanks for spotting this issue, however I got another fix, committed here: 1f5327a

thanks

@php-pulls

Comment on behalf of laruence at php.net:

another fix has beend committed, closed

@php-pulls php-pulls closed this Jun 1, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment