diff --git a/gtsam.h b/gtsam.h index 4a2cbb66c..5661cb520 100644 --- a/gtsam.h +++ b/gtsam.h @@ -36,8 +36,7 @@ * Namespace definitions * - Names of namespaces must start with a lowercase letter * - start a namespace with "namespace {" - * - end a namespace with exactly "}///\namespace [namespace_name]", optionally adding the name of the namespace - * - This ending is not C++ standard, and must contain "}///\namespace" to parse + * - end a namespace with exactly "}" * - Namespaces can be nested * Namespace usage * - Namespaces can be specified for classes in arguments and return values @@ -1363,7 +1362,7 @@ virtual class GenericProjectionFactor : gtsam::NonlinearFactor { typedef gtsam::GenericProjectionFactor GenericProjectionFactorCal3_S2; typedef gtsam::GenericProjectionFactor GenericProjectionFactorCal3DS2; -}///\namespace gtsam +} //\namespace gtsam //************************************************************************* // Pose2SLAM @@ -1418,7 +1417,7 @@ class Graph { gtsam::Marginals marginals(const pose2SLAM::Values& solution) const; }; -}///\namespace pose2SLAM +} //\namespace pose2SLAM //************************************************************************* // Pose3SLAM @@ -1473,7 +1472,7 @@ class Graph { gtsam::Marginals marginals(const pose3SLAM::Values& solution) const; }; -}///\namespace pose3SLAM +} //\namespace pose3SLAM //************************************************************************* // planarSLAM @@ -1561,7 +1560,7 @@ class Odometry { const gtsam::Ordering& ordering) const; }; -}///\namespace planarSLAM +} //\namespace planarSLAM //************************************************************************* // VisualSLAM @@ -1686,7 +1685,7 @@ class LevenbergMarquardtOptimizer { visualSLAM::Values values() const; }; -}///\namespace visualSLAM +} //\namespace visualSLAM //************************************************************************ // sparse BA @@ -1767,5 +1766,5 @@ class LevenbergMarquardtOptimizer { sparseBA::Values optimizeSafely(); sparseBA::Values values() const; }; -}///\namespace sparseBA +} //\namespace sparseBA diff --git a/wrap/Module.cpp b/wrap/Module.cpp index 241371b0b..757241ca2 100644 --- a/wrap/Module.cpp +++ b/wrap/Module.cpp @@ -295,8 +295,7 @@ Module::Module(const string& interfacePath, >> namespace_name_p[push_back_a(namespaces)] >> ch_p('{') >> *(include_p | class_p | templateSingleInstantiation_p | global_function_p | namespace_def_p | comments_p) - >> str_p("}///\\namespace") // end namespace, avoid confusion with classes // FIXME: check for absense of semicolon to disambiguate - >> !namespace_name_p) + >> ch_p('}')) [pop_a(namespaces)]; Rule using_namespace_p = diff --git a/wrap/tests/testNamespaces.h b/wrap/tests/testNamespaces.h index 68b6e041e..2c12686c0 100644 --- a/wrap/tests/testNamespaces.h +++ b/wrap/tests/testNamespaces.h @@ -17,7 +17,7 @@ class ClassB { // check namespace handling Vector aGlobalFunction(); -}///\namespace ns1 +} #include namespace ns2 { @@ -38,7 +38,7 @@ class ClassB { ClassB(); }; -}///\namespace ns3 +} class ClassC { ClassC(); @@ -47,7 +47,7 @@ class ClassC { // separate namespace global function, same name Vector aGlobalFunction(); -}///\namespace ns2 +} //\namespace ns2 class ClassD { ClassD();