Tweak to reporter registrar to make sure generated name is unique

catch_reporter_basic.hpp

@@ -170,6 +170,8 @@ namespace Catch
         bool m_firstSectionInTestCase;
     };
 
+    CATCH_REGISTER_REPORTER( "basic", BasicReporter );
+    
 } // end namespace Catch
     
 #endif // TWOBLUECUBES_CATCH_REPORTER_BASIC_HPP_INCLUDED

catch_reporter_junit.hpp

@@ -255,6 +255,8 @@ namespace Catch
         std::ostringstream m_stdErr;
     };
 
+    CATCH_REGISTER_REPORTER( "junit", JunitReporter );
+    
 } // end namespace Catch
 
 #endif // TWOBLUECUBES_CATCH_REPORTER_JUNIT_HPP_INCLUDED

catch_reporter_xml.hpp

@@ -150,6 +150,8 @@ namespace Catch
         XmlWriter m_xml;
     };
 
+    CATCH_REGISTER_REPORTER( "xml", XmlReporter );
+
 } // end namespace Catch
 
 #endif // TWOBLUECUBES_CATCH_REPORTER_XML_HPP_INCLUDED

catch_runner.hpp

@@ -23,10 +23,6 @@ namespace Catch
 {
     inline int Main( int argc, char * const argv[] )
     {
-        ReporterRegistry::instance().registerReporter<BasicReporter>( "basic" );
-        ReporterRegistry::instance().registerReporter<XmlReporter>( "xml" );
-        ReporterRegistry::instance().registerReporter<JunitReporter>( "junit" );
-
         RunnerConfig config;
         ArgParser( argc, argv, config );     
         

internal/catch_commandline.hpp

@@ -18,6 +18,10 @@
 
 namespace Catch
 {
+    // !TBD: This could be refactored to be more "declarative"
+    //       have a table up front that relates the mode, option strings, # arguments, names of arguments
+    //       - may not be worth it at this scale
+    
     // -l, --list tests [xml] lists available tests (optionally in xml)
     // -l, --list reporters [xml] lists available reports (optionally in xml)
     // -l, --list all [xml] lists available tests and reports (optionally in xml)

internal/catch_reporter_registry.hpp

@@ -180,6 +180,6 @@ namespace Catch
     }; 
 }
 
-#define CATCH_REGISTER_REPORTER( name, reporterType ) Catch::ReporterRegistrar<reporterType> INTERNAL_CATCH_UNIQUE_NAME(  catch_internal_ReporterReg )( name );
+#define CATCH_REGISTER_REPORTER( name, reporterType ) Catch::ReporterRegistrar<reporterType> catch_internal_RegistrarFor##reporterType( name );
 
 #endif // TWOBLUECUBES_CATCH_REPORTER_REGISTRY_HPP_INCLUDED