c++: minor stylistic changes

* data/c++m4: Remove useless comment lines.
* data/variant.hh (self_type): Use this typedef instead of variant<S>.
(b4_symbol_constructor_define_): Remove commented-out line, and stylistic
change (avoid blank line).

diff --git a/data/c++.m4 b/data/c++.m4
index 87bc5e0dcf184ab5fc3bea00154c8ac8bbe36bab..14eb42acf0ac8427dd7247f2194b7b64c27f4cdd 100644 (file)
--- a/data/c++.m4
+++ b/data/c++.m4
@@ -234,7 +234,6 @@ m4_define([b4_public_types_declare],
       enum { empty = 0 };
 
       /// The symbol type.
       enum { empty = 0 };
 
       /// The symbol type.

-      ///

       /// -1 when this symbol is empty.
       token_number_type type;
     };
       /// -1 when this symbol is empty.
       token_number_type type;
     };

diff --git a/data/variant.hh b/data/variant.hh
index da0d2a394091a81a2136c38ed1eb8bf4832ebe23..84ea7798b20700ad5a5a4edfdd2b6706fedce3a2 100644 (file)
--- a/data/variant.hh
+++ b/data/variant.hh
@@ -173,7 +173,7 @@ m4_define([b4_variant_define],
     /// variant::move ().
     template <typename T>
     void
     /// variant::move ().
     template <typename T>
     void

-    swap (variant<S>& other)
+    swap (self_type& other)

     {]b4_parse_assert_if([
       YYASSERT (built);
       YYASSERT (other.built);
     {]b4_parse_assert_if([
       YYASSERT (built);
       YYASSERT (other.built);


@@ -186,7 +186,7 @@ m4_define([b4_variant_define],
     /// Destroys \a other.
     template <typename T>
     void
     /// Destroys \a other.
     template <typename T>
     void

-    move (variant<S>& other)
+    move (self_type& other)

     {]b4_parse_assert_if([
       YYASSERT (! built);])[
       build<T>();
     {]b4_parse_assert_if([
       YYASSERT (! built);])[
       build<T>();


@@ -197,7 +197,7 @@ m4_define([b4_variant_define],
     /// Copy the content of \a other to this.
     template <typename T>
     void
     /// Copy the content of \a other to this.
     template <typename T>
     void

-    copy (const variant<S>& other)
+    copy (const self_type& other)

     {
       build<T> (other.as<T> ());
     }
     {
       build<T> (other.as<T> ());
     }


@@ -314,9 +314,8 @@ b4_join(b4_symbol_if([$1], [has_type],
         b4_locations_if([const location_type& l])))[
   {
     symbol_type res (token::]b4_symbol([$1], [id])[]b4_locations_if([, l])[);
         b4_locations_if([const location_type& l])))[
   {
     symbol_type res (token::]b4_symbol([$1], [id])[]b4_locations_if([, l])[);

-    ]b4_symbol_if([$1], [has_type], [res.value.build (v);])[
-    //    ]b4_locations_if([res.location = l;])[
-    return res;
+    ]b4_symbol_if([$1], [has_type], [res.value.build (v);
+    ])[return res;

   }
 
 ]])])])
   }
 
 ]])])])