bison-patches
[Top][All Lists]
Advanced

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

[PATCH 0/8] {master} revamp the handling of -W


From: Akim Demaille
Subject: [PATCH 0/8] {master} revamp the handling of -W
Date: Thu, 14 Feb 2013 15:11:04 +0100

This series of patches address a few shortcomings about the handling
of warning categories:

- Too much duplication.

- Hairy code.

- Théophile pointed out that supports for -Werror=foo has leaked
  to -r and -f (and -ferror=caret, nonsensical, is accepted).

- Joel pointed out that it would be nice to be able to enable
  -Wempty-rule when %empty is used, except if -Wno-empty-rule is
  set, which requires to tell the difference between a disabled
  warning (-Wno-empty-rule), and a warning that was not enabled.

I'd be happy to hear comments.  I shall apply these in a couple
of days.  Then I'll proceed on %empty.


Akim Demaille (8):
  options: no longer document warnings when diagnosing an invalid -W
  diagnostics: no longer pretty-print rules in error messages, carets
    suffice
  diagnostics: factor the list of warning names
  options: don't accept "error=" for -f and -r
  options: simplify the handling of -W
  diagnostics: revamp the handling of -Werror
  style: more argument handling of -W into the diagnostics module
  getargs: minor simplification

 NEWS                |  23 ++++++
 TODO                |  23 ------
 src/complain.c      | 229 +++++++++++++++++++++++++++++++++++++++++++---------
 src/complain.h      |  66 +++++++++++----
 src/getargs.c       |  87 ++++----------------
 src/getargs.h       |   1 +
 src/gram.c          |  33 ++------
 src/main.c          |   1 +
 src/muscle-tab.c    |   2 +-
 tests/conflicts.at  |  30 +++----
 tests/existing.at   |   2 +-
 tests/input.at      |  23 ++++++
 tests/reduce.at     |  38 +--------
 tests/regression.at |   4 +-
 14 files changed, 331 insertions(+), 231 deletions(-)

-- 
1.8.1.3




reply via email to

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