[PATCH] Better macro error messages involving initializer lists

Richard Trieu rtrieu at google.com
Fri Jul 19 14:41:47 PDT 2013


  Refactoring the earlier patch to take into account comments.

  - Instead of attempting a correction after every failure, only try after a "too many arguments" failure.
  - Rewrite the code that generates a new vector of token arguments to use the old vector of tokens.  This cuts back the amount of token copying to only when a correction is attempted.
  - Give a note on all arguments started with a brace list instead of only when the brace list is the entire token.

http://llvm-reviews.chandlerc.com/D986

CHANGE SINCE LAST DIFF
  http://llvm-reviews.chandlerc.com/D986?vs=2432&id=2920#toc

Files:
  test/Preprocessor/macro_fn.c
  test/Preprocessor/macro_with_initializer_list.cpp
  include/clang/Basic/DiagnosticLexKinds.td
  lib/Lex/PPMacroExpansion.cpp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D986.2.patch
Type: text/x-patch
Size: 18610 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130719/31af291a/attachment.bin>


More information about the cfe-commits mailing list