slang-reflect: Minor improvements of the code

* Make use of std::views:keys * Fix typo in a comment
MikePopoloski · Nov 13, 2024 · 5890207 · 5890207
1 parent 26f66eb
commit 5890207
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 4 deletions.
diff --git a/tools/reflect/src/SvEnum.cpp b/tools/reflect/src/SvEnum.cpp
@@ -4,7 +4,6 @@
 #include "SvEnum.h"
 
 #include <fmt/format.h>
-#include <iostream>
 
 void SvEnum::toCpp(HppFile& hppFile, std::string_view, const SvAliases&, bool) const {
     auto underlyingType = [&] {
@@ -77,7 +76,7 @@ void SvEnum::toCpp(HppFile& hppFile, std::string_view, const SvAliases&, bool) c
     hppFile.increaseIndent();
     hppFile.addWithIndent("switch (__data.type) {\n");
     hppFile.increaseIndent();
-    for (const auto& [name, value] : members)
+    for (const auto& name : members | std::views::keys)
         hppFile.addWithIndent(fmt::format("case Type::{0}: os << \"{0}\"; break;\n", name));
     hppFile.decreaseIndent();
     hppFile.addWithIndent("}\n");

diff --git a/tools/reflect/src/SvStruct.cpp b/tools/reflect/src/SvStruct.cpp
@@ -176,7 +176,7 @@ void SvStruct::toCpp(HppFile& hppFile, const std::string_view _namespace, const
         hppFile.addWithIndent(fmt::format("operator {}() const {{\n", cppTypeStr));
         hppFile.increaseIndent();
         hppFile.addWithIndent(fmt::format("{} ret = 0;\n", cppTypeStr));
-        for (const auto& [name, _] : members) {
+        for (const auto& name : members | std::views::keys) {
             hppFile.addWithIndent(
                 fmt::format("ret |= static_cast<{0}>({1}) << {1}_s;\n", cppTypeStr, name));
         }

diff --git a/tools/reflect/src/SvUnion.cpp b/tools/reflect/src/SvUnion.cpp
@@ -36,7 +36,7 @@ void SvUnion::toCpp(HppFile& hppFile, std::string_view _namespace, const SvAlias
     // Create a sc bit vector to store the data of the Union
     hppFile.addWithIndent(fmt::format("{} union_data;\n\n", cppTypeStr));
 
-    // Check if some headers needs to be included
+    // Check if some headers need to be included
     for (const auto& member : type.getCanonicalType().as<slang::ast::Scope>().members()) {
         const auto& variable = member.as<slang::ast::VariableSymbol>();