[Boost-commit] svn:boost r81273 - in trunk: boost/wave/util libs/wave

Subject: [Boost-commit] svn:boost r81273 - in trunk: boost/wave/util libs/wave
From: hartmut.kaiser_at_[hidden]
Date: 2012-11-09 20:20:19

Author: hkaiser
Date: 2012-11-09 20:20:19 EST (Fri, 09 Nov 2012)
New Revision: 81273
URL: http://svn.boost.org/trac/boost/changeset/81273

Log:
Wave: Fixed a problem with context<>::add_macro_definition which sometimes appended a superfluous T_EOF to the macro replacement list.
Text files modified: 
   trunk/boost/wave/util/cpp_iterator.hpp |     9 +++++----                               
   trunk/libs/wave/ChangeLog              |    31 +++++++++++++++++--------------         
   2 files changed, 22 insertions(+), 18 deletions(-)

Modified: trunk/boost/wave/util/cpp_iterator.hpp
==============================================================================
--- trunk/boost/wave/util/cpp_iterator.hpp	(original)
+++ trunk/boost/wave/util/cpp_iterator.hpp	2012-11-09 20:20:19 EST (Fri, 09 Nov 2012)
@@ -202,13 +202,14 @@
     boost::wave::util::retrieve_macrodefinition(*hit.trees.begin(),
         BOOST_WAVE_MACRO_DEFINITION_ID, macrodefinition, act_pos, true);
 
+    // get rid of trailing T_EOF
+    if (!macrodefinition.empty() && token_id(macrodefinition.back()) == T_EOF)
+        macrodefinition.resize(macrodefinition.size()-1);
+
 //  If no macrodefinition is given, and the macro string does not end with a
 //  '=', then the macro should be defined with the value '1'
-    if (0 == macrodefinition.size() &&
-        '=' != macrostring[macrostring.size()-1])
-    {
+    if (macrodefinition.empty() && '=' != macrostring[macrostring.size()-1])
         macrodefinition.push_back(token_type(T_INTLIT, "1", act_pos));
-    }
 
 // add the new macro to the macromap
     return ctx.add_macro_definition(macroname, has_parameters, macroparameters,

Modified: trunk/libs/wave/ChangeLog
==============================================================================
--- trunk/libs/wave/ChangeLog	(original)
+++ trunk/libs/wave/ChangeLog	2012-11-09 20:20:19 EST (Fri, 09 Nov 2012)
@@ -20,22 +20,25 @@
 
 CHANGELOG
 
-- Added util::create_directories() wrapper to account for new behavior of
-  boost::filesystem::create_directories().
-- Fixed an obscure problem when preprocessing directives wouldn't be recognized
-  if the previous line contained nothing but an empty macro invocation
-  (see new test case t_9_023.cpp)
-- Added a new command line option --license=<file> to the Wave driver tool
-  which allows to pre-pend the content of a (license) file to every newly
-  created file. This option simplifies the implementation of partial
-  preprocessing as done on Phoenix, Fusion, etc.
-- Changed the effect of the -N command line option to simply not expand
-  the macro by pretending it was not defined. Earlier the whole invocation
-  sequence was skipped, now we skip only the macro itself, which still expands
-  the arguments for the macro invocation.
+Boost V1.53:
+ - Fixed a problem with context<>::add_macro_definition which sometimes 
+   appended a superfluous T_EOF to the macro replacement list.
 
 Boost V1.52.0:
- - Fixed a couple of compilation warnings
+ - Added util::create_directories() wrapper to account for new behavior of
+   boost::filesystem::create_directories().
+ - Fixed an obscure problem when preprocessing directives wouldn't be recognized
+   if the previous line contained nothing but an empty macro invocation
+   (see new test case t_9_023.cpp)
+ - Added a new command line option --license=<file> to the Wave driver tool
+   which allows to pre-pend the content of a (license) file to every newly
+   created file. This option simplifies the implementation of partial
+   preprocessing as done on Phoenix, Fusion, etc.
+ - Changed the effect of the -N command line option to simply not expand
+   the macro by pretending it was not defined. Earlier the whole invocation
+   sequence was skipped, now we skip only the macro itself, which still expands
+   the arguments for the macro invocation.
+  - Fixed a couple of compilation warnings
 
 Boost V1.51.0
  - Fixed #7050: Invalid memory write bug in lexing_exception