Hi, Nuno, Thanks for reminding, I've attached a patch for this. could you take a look? GET_REFLECTION_OBJECT_PTR used widely, but define yet another macro seems useless, so I expand the macro.
Thanks. Best -- reeze | reeze.cn 在 2012年5月23日星期三,上午2:50,Nuno Lopes 写道: > > Commit: 054f3e3ce5af13c2c3a6ccd54f7dc3e2f6cd4f74 > > Author: reeze <reeze....@gmail.com (mailto:reeze....@gmail.com)> Tue, 3 Apr > > 2012 13:47:16 +0800 > > Parents: 3bf53aa911e1e2128a11aee45c126000635de006 > > Branches: PHP-5.4 master > > > > Link: > > http://git.php.net/?p=php-src.git;a=commitdiff;h=054f3e3ce5af13c2c3a6ccd54f7dc3e2f6cd4f74 > > > > Log: > > Implemented FR #61602 Allow access to name of constant used as default value > > > > Bugs: > > https://bugs.php.net/61602 > > > > diff --git a/ext/reflection/php_reflection.c > > b/ext/reflection/php_reflection.c > > index 1cf65ce..ef1ed7e 100644 > > --- a/ext/reflection/php_reflection.c > > +++ b/ext/reflection/php_reflection.c > > @@ -1457,6 +1457,57 @@ static void > > _reflection_export(INTERNAL_FUNCTION_PARAMETERS, zend_class_entry *c > > } > > /* }}} */ > > +/* {{{ _reflection_param_get_default_param */ > > +static parameter_reference > > *_reflection_param_get_default_param(INTERNAL_FUNCTION_PARAMETERS) > > +{ > > + reflection_object *intern; > > + parameter_reference *param; > > + > > + if (zend_parse_parameters_none() == FAILURE) { > > + return NULL; > > + } > > + > > + GET_REFLECTION_OBJECT_PTR(param); > > > > > You cannot use GET_REFLECTION_OBJECT_PTR here because it has a > 'return;' statement (i.e., without a return value). While gcc accepts > this (but issues a warning), it's considered an error in C99, and it > breaks the build with other compilers. > Please either fix it or revert the patch. > > Nuno
-- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php