You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@thrift.apache.org by dr...@apache.org on 2010/08/26 23:49:46 UTC

svn commit: r989948 - in /incubator/thrift/trunk/compiler/cpp/src: generate/t_generator.h generate/t_generator_registry.h parse/t_program.h

Author: dreiss
Date: Thu Aug 26 21:49:45 2010
New Revision: 989948

URL: http://svn.apache.org/viewvc?rev=989948&view=rev
Log:
Clean up some style from r987565

Modified:
    incubator/thrift/trunk/compiler/cpp/src/generate/t_generator.h
    incubator/thrift/trunk/compiler/cpp/src/generate/t_generator_registry.h
    incubator/thrift/trunk/compiler/cpp/src/parse/t_program.h

Modified: incubator/thrift/trunk/compiler/cpp/src/generate/t_generator.h
URL: http://svn.apache.org/viewvc/incubator/thrift/trunk/compiler/cpp/src/generate/t_generator.h?rev=989948&r1=989947&r2=989948&view=diff
==============================================================================
--- incubator/thrift/trunk/compiler/cpp/src/generate/t_generator.h (original)
+++ incubator/thrift/trunk/compiler/cpp/src/generate/t_generator.h Thu Aug 26 21:49:45 2010
@@ -67,7 +67,7 @@ class t_generator {
 
   /**
    * check whether sub-namespace declaraction is used by generator.
-   * e.g. allow 
+   * e.g. allow
    * namespace py.twisted bar
    * to specify namespace to use when -gen py:twisted is specified.
    * Will be called with subnamespace, i.e. is_valid_namespace("twisted")

Modified: incubator/thrift/trunk/compiler/cpp/src/generate/t_generator_registry.h
URL: http://svn.apache.org/viewvc/incubator/thrift/trunk/compiler/cpp/src/generate/t_generator_registry.h?rev=989948&r1=989947&r2=989948&view=diff
==============================================================================
--- incubator/thrift/trunk/compiler/cpp/src/generate/t_generator_registry.h (original)
+++ incubator/thrift/trunk/compiler/cpp/src/generate/t_generator_registry.h Thu Aug 26 21:49:45 2010
@@ -74,7 +74,7 @@ class t_generator_factory_impl : public 
     return new generator(program, parsed_options, option_string);
   }
 
-  bool is_valid_namespace(const std::string& sub_namespace){
+  virtual bool is_valid_namespace(const std::string& sub_namespace) {
     return generator::is_valid_namespace(sub_namespace);
   }
 };

Modified: incubator/thrift/trunk/compiler/cpp/src/parse/t_program.h
URL: http://svn.apache.org/viewvc/incubator/thrift/trunk/compiler/cpp/src/parse/t_program.h?rev=989948&r1=989947&r2=989948&view=diff
==============================================================================
--- incubator/thrift/trunk/compiler/cpp/src/parse/t_program.h (original)
+++ incubator/thrift/trunk/compiler/cpp/src/parse/t_program.h Thu Aug 26 21:49:45 2010
@@ -178,7 +178,7 @@ class t_program : public t_doc {
       if(! it->second->is_valid_namespace(sub_namespace)) {
         throw base_language +" generator does not accept '" + sub_namespace + "' as sub-namespace!";
       }
-    } 
+    }
 
     namespaces_[language] = name_space;
   }