Hey: the reason I am not sure about that is, the YYPARSE_PARAM is an bison specific macro, so it's a little weird if we define them in another header file...
thanks On Fri, Apr 5, 2013 at 11:13 PM, Laruence <larue...@php.net> wrote: > Hey: > I found another way, which will not break bison 2.3 support. > > that is, move the YYPARSE_PARAM definition into zend.h, or other > header file which will always include before zend_language_parser.h and > also include by zend_language_parse.c > > what do you think? > > thanks > > > On Thu, Apr 4, 2013 at 9:24 PM, Johannes Schlüter > <johan...@schlueters.de>wrote: > >> On Wed, 2013-04-03 at 19:04 +0800, Laruence wrote: >> > Hey: >> > remi did, see https://bugs.php.net/bug.php?id=64503 >> > >> > but seems start from 2.4.1 >> >> Then please update Zend/acinclude.m4: accordingly. >> >> johannes >> >> >> > > > -- > Laruence Xinchen Hui > http://www.laruence.com/ > -- Laruence Xinchen Hui http://www.laruence.com/