Hey: On Fri, Dec 26, 2014 at 7:54 PM, Andrea Faulds <a...@ajf.me> wrote: > >> On 26 Dec 2014, at 05:57, Levi Morrison <le...@php.net> wrote: >> >>> On Thu, Dec 25, 2014 at 2:33 PM, Michael Wallner <m...@php.net> wrote: >>> There's already ZEND_RESULT_CODE, or did I miss anything? >> >> According to lxr.php.net, this symbol ZEND_RESULT_CODE is not >> referenced in any place except its definition. We can begin using it >> if we like, or even rename it. Theoretically renaming it may break >> extensions; none that I could find reference it, though. > > Hey Levi, > > I think we should just rename it, it's a little too long. zend_status or > ZEND_STATUS would be fine with me, preferably the former since we tend to use > lowercase type names. Another possibility might be zend_result. > zend_status +1 > If you folks like, I could go ahead and write a patch for master to rename it > and use it in a bunch of places.
great, thanks... please only added it to these functions who already use success/failure, don't change others :) thanks > > Thanks. > > -- > Andrea Faulds > http://ajf.me/ > -- Xinchen Hui @Laruence http://www.laruence.com/ -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
