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 87bc5e0..14eb42a 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.
-      ///
       /// -1 when this symbol is empty.
       token_number_type type;
     };

diff --git a/data/variant.hh b/data/variant.hh
index da0d2a3..84ea779 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
-    swap (variant<S>& other)
+    swap (self_type& other)
     {]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
-    move (variant<S>& other)
+    move (self_type& other)
     {]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 (const variant<S>& other)
+    copy (const self_type& other)
     {
       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_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;
   }
 
 ]])])])