[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers
From: |
Stefano Lattarini |
Subject: |
Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers |
Date: |
Sat, 14 Jul 2012 10:03:32 +0200 |
Hi Akim.
On 07/14/2012 09:24 AM, Akim Demaille wrote:
>
> Le 14 juil. 2012 à 09:13, Akim Demaille a écrit :
>
>> The failure is due to the input:
>>
>> %{
>> int yylex () { return 0; }
>> void yyerror (const char *s) { return; }
>> %}
>> %%
>> foobar : 'f' 'o' 'o' 'b' 'a' 'r' {};
>>
>> %{%} goes into the header when there is one, and it also goes
>> in the implementation file, so, since glr.c include its header,
>> you get two definitions.
>
> No, that's not it, I overlooked the test case. The issue
> is simpler: the header file contains the definition of these
> guys, and it is included in two different compilation units.
>
> Attached is an updated patch.
>
>> I don't see it since 2.6 has several improvements in this area.
>
> I don't see it because this section does not appear in the
> header file since the superior %code approach was added.
>
Thanks for working this out. Feel free to push it out to yacc-work;
I'll give that branch more testing today or tomorrow (on Solaris and
BSD systems), then we'll be able to merge it into maint.
Thanks,
Stefano
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, (continued)
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Stefano Lattarini, 2012/07/13
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Akim Demaille, 2012/07/13
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Stefano Lattarini, 2012/07/13
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Akim Demaille, 2012/07/13
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Stefano Lattarini, 2012/07/13
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Stefano Lattarini, 2012/07/13
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Akim Demaille, 2012/07/13
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Stefano Lattarini, 2012/07/13
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Akim Demaille, 2012/07/14
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Akim Demaille, 2012/07/14
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers,
Stefano Lattarini <=
- [PATCH 0/4] ylwrap: support C++ and others that generate several files, Akim Demaille, 2012/07/14
- [PATCH 1/4] tests: upgrade and fix Bison test case, Akim Demaille, 2012/07/14
- Re: [PATCH 1/4] tests: upgrade and fix Bison test case, Stefano Lattarini, 2012/07/14
- Re: [PATCH 1/4] tests: upgrade and fix Bison test case, Akim Demaille, 2012/07/14
- Re: [PATCH 1/4] tests: upgrade and fix Bison test case, Stefano Lattarini, 2012/07/14
- Re: [PATCH 1/4] tests: upgrade and fix Bison test case, Akim Demaille, 2012/07/14
- [PATCH 2/4] ylwrap: refactoring: don't rely on the file order, Akim Demaille, 2012/07/14
- [PATCH 3/4] ylwrap: refactor: move loop invariant, Akim Demaille, 2012/07/14
- [PATCH 4/4] ylwrap: fix C++ support, Akim Demaille, 2012/07/14
- Re: [PATCH 4/4] ylwrap: fix C++ support, Stefano Lattarini, 2012/07/14