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;
}