I'll double check later today, it might be that this comes in from the startup phase and we can't do zend_error() for some reason. Either way I'll get it merged soonish. Thanks. -Sara
On Tue, Jun 20, 2017 at 6:53 AM, François Laupretre <franc...@tekwire.net> wrote: > Hi Sara, > > Le 19/06/2017 à 23:33, Sara Golemon a écrit : >> >> I was about to merge this, but ran into some issues (mostly minor). >> Could you at least address the fwrite(stderr, ...) item (and >> preferably clean up the style nits while you're at it)? >> >> Oh, and I forgot to include it in the CR, but there are some >> tabs/spaces issues as well. > > I fixed typos and style issues. > > About the 'fwrite(stderr,', I just used the same mechanism used by > zend_load_extension() to raise an error. If you consider it is safe to > replace it with a call to zend_error(), I'll do it. > > Regards > > François -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php