[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: |
Akim Demaille |
Subject: |
Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers |
Date: |
Sat, 14 Jul 2012 09:24:08 +0200 |
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.
0001-tests-fix-bison-input-file.patch
Description: Binary data
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, (continued)
- 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 <=
- Re: [PATCH 3/3] ylwrap: rename header inclusion in generated parsers, Stefano Lattarini, 2012/07/14
- [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