diff --git a/src/ZoneCodeGeneratorLib/Generating/CodeGenerator.cpp b/src/ZoneCodeGeneratorLib/Generating/CodeGenerator.cpp index a501e99c..b1ca0f86 100644 --- a/src/ZoneCodeGeneratorLib/Generating/CodeGenerator.cpp +++ b/src/ZoneCodeGeneratorLib/Generating/CodeGenerator.cpp @@ -7,6 +7,7 @@ #include "Templates/ZoneWriteTemplate.h" #include +#include #include #include @@ -41,7 +42,7 @@ bool CodeGenerator::GenerateCodeForTemplate(RenderingContext* context, ICodeTemp if (!stream.is_open()) { - std::cout << "Failed to open file '" << p.string() << "'\n"; + std::cerr << std::format("Failed to open file '{}'\n", p.string()); return false; } @@ -58,7 +59,7 @@ bool CodeGenerator::GetAssetWithName(IDataRepository* repository, const std::str auto* def = repository->GetDataDefinitionByName(name); if (def == nullptr) { - std::cout << "Could not find type with name '" << name << "'\n"; + std::cerr << std::format("Could not find type with name '{}'\n", name); return false; } @@ -66,13 +67,13 @@ bool CodeGenerator::GetAssetWithName(IDataRepository* repository, const std::str auto* info = defWithMembers != nullptr ? repository->GetInformationFor(defWithMembers) : nullptr; if (info == nullptr) { - std::cout << "Could not find type with name '" << name << "'\n"; + std::cerr << std::format("Could not find type with name '{}'\n", name); return false; } if (!StructureComputations(info).IsAsset()) { - std::cout << "Type is not an asset '" << name << "'\n"; + std::cerr << std::format("Type is not an asset '{}'\n", name); return false; } @@ -100,7 +101,7 @@ bool CodeGenerator::GenerateCode(IDataRepository* repository) const auto foundTemplate = m_template_mapping.find(templateName); if (foundTemplate == m_template_mapping.end()) { - std::cout << "Unknown template '" << generationTask.m_template_name << "'.\n"; + std::cerr << std::format("Unknown template '{}'.\n", generationTask.m_template_name); return false; } @@ -111,13 +112,13 @@ bool CodeGenerator::GenerateCode(IDataRepository* repository) auto context = RenderingContext::BuildContext(repository, asset); if (!GenerateCodeForTemplate(context.get(), foundTemplate->second.get())) { - std::cout << "Failed to generate code for asset '" << asset->m_definition->GetFullName() << "' with preset '" << foundTemplate->first - << "'\n"; + std::cerr << std::format( + "Failed to generate code for asset '{}' with preset '{}'\n", asset->m_definition->GetFullName(), foundTemplate->first); return false; } - std::cout << "Successfully generated code for asset '" << asset->m_definition->GetFullName() << "' with preset '" << foundTemplate->first - << "'\n"; + std::cout << std::format( + "Successfully generated code for asset '{}' with preset '{}'\n", asset->m_definition->GetFullName(), foundTemplate->first); } } else @@ -133,9 +134,7 @@ bool CodeGenerator::GenerateCode(IDataRepository* repository) } const auto end = std::chrono::steady_clock::now(); if (m_args->m_verbose) - { - std::cout << "Generating code took " << std::chrono::duration_cast(end - start).count() << "ms\n"; - } + std::cout << std::format("Generating code took {}ms\n", std::chrono::duration_cast(end - start).count()); return true; } diff --git a/src/ZoneCodeGeneratorLib/Generating/CodeGenerator.h b/src/ZoneCodeGeneratorLib/Generating/CodeGenerator.h index 8c24f10f..b05bd382 100644 --- a/src/ZoneCodeGeneratorLib/Generating/CodeGenerator.h +++ b/src/ZoneCodeGeneratorLib/Generating/CodeGenerator.h @@ -21,6 +21,5 @@ private: static bool GetAssetWithName(IDataRepository* repository, const std::string& name, StructureInformation*& asset); const ZoneCodeGeneratorArguments* m_args; - std::unordered_map> m_template_mapping; }; diff --git a/src/ZoneCodeGeneratorLib/Generating/ICodeTemplate.h b/src/ZoneCodeGeneratorLib/Generating/ICodeTemplate.h index a9c449af..833d9900 100644 --- a/src/ZoneCodeGeneratorLib/Generating/ICodeTemplate.h +++ b/src/ZoneCodeGeneratorLib/Generating/ICodeTemplate.h @@ -9,14 +9,14 @@ class CodeTemplateFile { public: - std::string m_file_name; - int m_tag; - CodeTemplateFile(std::string fileName, const int tag) : m_file_name(std::move(fileName)), m_tag(tag) { } + + std::string m_file_name; + int m_tag; }; class ICodeTemplate diff --git a/src/ZoneCodeGeneratorLib/Generating/Templates/ZoneLoadTemplate.cpp b/src/ZoneCodeGeneratorLib/Generating/Templates/ZoneLoadTemplate.cpp index d088da67..6493a0cd 100644 --- a/src/ZoneCodeGeneratorLib/Generating/Templates/ZoneLoadTemplate.cpp +++ b/src/ZoneCodeGeneratorLib/Generating/Templates/ZoneLoadTemplate.cpp @@ -3,6 +3,7 @@ #include "Domain/Computations/MemberComputations.h" #include "Domain/Computations/StructureComputations.h" #include "Internal/BaseTemplate.h" +#include "Utils/StringUtils.h" #include #include @@ -1349,8 +1350,7 @@ std::vector ZoneLoadTemplate::GetFilesToRender(RenderingContex std::vector files; auto assetName = context->m_asset->m_definition->m_name; - for (auto& c : assetName) - c = static_cast(tolower(c)); + utils::MakeStringLowerCase(assetName); files.emplace_back(std::format("{0}/{0}_load_db.h", assetName), TAG_HEADER); files.emplace_back(std::format("{0}/{0}_load_db.cpp", assetName), TAG_SOURCE);