From e85edc9ed5f31da291d8c16a5f1a044ff3c46c11 Mon Sep 17 00:00:00 2001 From: Andrew Haley Date: Thu, 3 Jul 2008 10:31:50 +0000 Subject: [PATCH] re PR preprocessor/33305 (We should warn about empty macro arguments) 2008-06-13 Andrew Haley PR preprocessor/33305 * macro.c (replace_args): Print a warning for empty macro arguments in C89 and C++. From-SVN: r137414 --- libcpp/ChangeLog | 6 ++++++ libcpp/macro.c | 11 +++++++++++ 2 files changed, 17 insertions(+) diff --git a/libcpp/ChangeLog b/libcpp/ChangeLog index 7f069414f7e..2d29a309566 100644 --- a/libcpp/ChangeLog +++ b/libcpp/ChangeLog @@ -1,3 +1,9 @@ +2008-06-13 Andrew Haley + + PR preprocessor/33305 + * macro.c (replace_args): Print a warning for empty macro + arguments in C89 and C++. + 2008-06-17 Ralf Wildenhues * Makefile.in ($(srcdir)/aclocal.m4): Update dependencies. diff --git a/libcpp/macro.c b/libcpp/macro.c index 016754bc952..edc2856551a 100644 --- a/libcpp/macro.c +++ b/libcpp/macro.c @@ -1009,6 +1009,17 @@ replace_args (cpp_reader *pfile, cpp_hashnode *node, cpp_macro *macro, macro_arg if (src->flags & PASTE_LEFT) paste_flag = dest - 1; } + else if (CPP_PEDANTIC (pfile) && ! macro->syshdr + && ! CPP_OPTION (pfile, c99) + && ! cpp_in_system_header (pfile)) + { + cpp_error (pfile, CPP_DL_PEDWARN, + "invoking macro %s argument %d: " + "empty macro arguments are undefined" + " in ISO C90 and ISO C++98", + NODE_NAME (node), + src->val.arg_no); + } /* Avoid paste on RHS (even case count == 0). */ if (!pfile->state.in_directive && !(src->flags & PASTE_LEFT))