Nikita Popov wrote: > On Wed, Jun 17, 2015 at 12:20 AM, Christoph Becker <cmbecke...@gmx.de> > wrote: > >> Johannes Schlüter wrote: >> >>> On Tue, 2015-06-16 at 02:02 +0200, Christoph Becker wrote: >>> >>>> Fair enough. However, the Windows sources neither contain the generated >>>> parser files, because they would break linking on Windows, see >>>> <http://news.php.net/php.internals/86685>. >>> >>> This is a bug and has to be addressed. >> >> I have investigated further. The issue is commit b8e2629[1]. I'm >> certain that Sara had good reasons to mark zendparse() as ZEND_API. >> However, that breaks building with the generated zend_language_parser >> files with MSVC, if they have been generated with a somewhat recent >> bison (2.4.1 still works, 2.7.12 fails), because newer versions declare >> zendparse() again in .c and .h – without ZEND_API. >> >> [1] >> < >> https://github.com/php/php-src/commit/b8e262901e56a4d9a6664d348c9878246579239e > > We need some ZEND_API way to invoke the parser from extensions. If it's > inconvenient to have on zendparse() directly because bison, I'd suggest > dropping it there and creating a ZEND_API wrapping function.
Thanks! That would be a simple and clean solution. I'll make a respective PR if no one beats me to it. -- Christoph M. Becker -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php
