[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[PATCH 0/9] {master} check the kind of %define variable values

From: Akim Demaille
Subject: [PATCH 0/9] {master} check the kind of %define variable values
Date: Mon, 15 Apr 2013 10:58:19 +0200

Hi all,

A bunch of patches to check and enforce the kind of the variables.
The documentation is catching up, but there are still things to
document and check (e.g., the handling of -D/-F).

I have chosen to use -Wdeprecated warnings even for variables introduced
in this release, as I have been spreading betas of it quite widely.
It would be unfair for early adopter, and it does not cost much to
maintain it this way.

Some of the messages are unpleasant, but I don't know if I'll spend
time on improving them.  For instance on

  %define namespace foo

you'll get

  /tmp/foo.y:1.9-17: warning: deprecated directive, use '%define api.namespace 
foo' [-Wdeprecated]
   %define namespace foo
  /tmp/foo.y:1.9-17: warning: %define variable 'api.namespace' requires '{...}' 
values [-Wdeprecated]
   %define namespace foo


  /tmp/foo.y:1.9-17: warning: deprecated directive, use '%define api.namespace 
{foo}' [-Wdeprecated]
   %define namespace foo

would be nicer.

Akim Demaille (9):
  muscle: factor conditionals on defined %define variables
  muscle: factor the kind check in M4
  muscle: factor the computation of variable names
  muscle: factor the handling of used variables
  muscle: factor the field retrieval
  muscle: minor simplification which uncovers a missing warning
  muscle: minor refactoring
  muscle: let -D/-F support the three kinds of %define variable values
  muscle: enforce definition syntax for keyword variables

 NEWS             |   2 +
 data/bison.m4    |  60 +++++++++++++++------
 data/c++.m4      |   6 +++
 doc/bison.texi   |  12 ++---
 src/getargs.c    |  29 ++++++++---
 src/muscle-tab.c | 155 ++++++++++++++++++++++++++++++++++++-------------------
 tests/    |  19 ++++---
 tests/   |  47 +++++++++++++++--
 8 files changed, 232 insertions(+), 98 deletions(-)


reply via email to

[Prev in Thread] Current Thread [Next in Thread]