Namespace now optional for general generator
authorJeremie St-Amand <jeremie.stamand@gmail.com>
Sun, 14 Jun 2015 16:56:48 +0000 (12:56 -0400)
committerJeremie St-Amand <jeremie.stamand@gmail.com>
Sun, 14 Jun 2015 16:56:48 +0000 (12:56 -0400)
If you have a schema with no namespace, the resulting C#/Java won't have
an erroneous empty namespace keyword.

src/idl_gen_general.cpp

index 9e34a2e..a212bf9 100644 (file)
@@ -875,11 +875,13 @@ static bool SaveClass(const LanguageParameters &lang, const Parser &parser,
   EnsureDirExists(namespace_dir);
 
   std::string code = "// automatically generated, do not modify\n\n";
-  code += lang.namespace_ident + namespace_general + lang.namespace_begin;
-  code += "\n\n";
+  if (!namespace_general.empty()) {
+         code += lang.namespace_ident + namespace_general + lang.namespace_begin;
+         code += "\n\n";
+  }
   if (needs_includes) code += lang.includes;
   code += classcode;
-  code += lang.namespace_end;
+  if (!namespace_general.empty()) code += lang.namespace_end;
   auto filename = namespace_dir + def.name + lang.file_extension;
   return SaveFile(filename.c_str(), code, false);
 }