[C#] Respect command line parameter for file name extension and suffix (#6779)
* Make CSharp use the command line parameter for file name extension and suffix * Default file name now has _generated in it, so we need to rename all the generated test files tests/namespace_test/NamespaceA/TableInC.* are no longer used, so these can be deleted * Fix C# testing projects so that they reference the new _generated.cs files * Revert "Fix C# testing projects so that they reference" This reverts commitcf62b9c5d4
. * Revert "Default file name now has _generated in it," This reverts commit042aa81f10
. * Revert "Make CSharp use the command line parameter for file name extension and suffix" This reverts commitba8cd06646
. * Make CSharp use the command line parameter for file name extension and suffix * Fixup clang tidy * Fix more clang-tidy
This commit is contained in:
parent
f20d2253ae
commit
2ca5f0e72b
|
@ -65,7 +65,7 @@ class CSharpGenerator : public BaseGenerator {
|
|||
one_file_code += enumcode;
|
||||
} else {
|
||||
if (!SaveType(enum_def.name, *enum_def.defined_namespace, enumcode,
|
||||
false))
|
||||
false, parser_.opts))
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
@ -81,14 +81,14 @@ class CSharpGenerator : public BaseGenerator {
|
|||
one_file_code += declcode;
|
||||
} else {
|
||||
if (!SaveType(struct_def.name, *struct_def.defined_namespace, declcode,
|
||||
true))
|
||||
true, parser_.opts))
|
||||
return false;
|
||||
}
|
||||
}
|
||||
|
||||
if (parser_.opts.one_file) {
|
||||
return SaveType(file_name_, *parser_.current_namespace_, one_file_code,
|
||||
true);
|
||||
true, parser_.opts);
|
||||
}
|
||||
return true;
|
||||
}
|
||||
|
@ -96,7 +96,8 @@ class CSharpGenerator : public BaseGenerator {
|
|||
// Save out the generated code for a single class while adding
|
||||
// declaration boilerplate.
|
||||
bool SaveType(const std::string &defname, const Namespace &ns,
|
||||
const std::string &classcode, bool needs_includes) const {
|
||||
const std::string &classcode, bool needs_includes,
|
||||
const IDLOptions &options) const {
|
||||
if (!classcode.length()) return true;
|
||||
|
||||
std::string code =
|
||||
|
@ -117,7 +118,10 @@ class CSharpGenerator : public BaseGenerator {
|
|||
}
|
||||
code += classcode;
|
||||
if (!namespace_name.empty()) { code += "\n}\n"; }
|
||||
auto filename = NamespaceDir(ns) + defname + ".cs";
|
||||
auto filename = NamespaceDir(ns) + defname;
|
||||
if (options.one_file) { filename += options.filename_suffix; }
|
||||
filename +=
|
||||
options.filename_extension.empty() ? ".cs" : options.filename_extension;
|
||||
return SaveFile(filename.c_str(), code, false);
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in New Issue