Updates to match API253011

DiligentGraphics · Jul 24, 2023 · 062a178 · 062a178
1 parent e1661f6
commit 062a178
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 10 deletions.
diff --git a/RenderStateNotation/scripts/cxx_config.py b/RenderStateNotation/scripts/cxx_config.py
@@ -60,7 +60,6 @@
 
     "ShaderDesc",
     "ShaderMacro",
-    "ShaderMacroArray",
     "ShaderResourceDesc",
     "ShaderCreateInfo",
 

diff --git a/RenderStateNotation/scripts/cxx_template.py b/RenderStateNotation/scripts/cxx_template.py
@@ -157,28 +157,27 @@
     Str = Allocator.CopyString(Json.get<std::string>());
 }
 
-template <>
-inline void WriteRSN(nlohmann::json& Json, const ShaderMacro* const pMacros, DynamicLinearAllocator& Allocator)
+inline void WriteRSN(nlohmann::json& Json, const ShaderMacroArray& Macros, DynamicLinearAllocator& Allocator)
 {
-    for (size_t i = 0; pMacros[i].Name != nullptr || pMacros[i].Definition != nullptr; i++)
+    for (size_t i = 0; Macros.Count; i++)
     {
         auto Object = nlohmann::json::object();
-        WriteRSN(Object, pMacros[i], Allocator);
+        WriteRSN(Object, Macros.Elements[i], Allocator);
         Json.push_back(Object);
     }
 }
 
-template <>
-inline void ParseRSN(const nlohmann::json& Json, const ShaderMacro*& pMacros, DynamicLinearAllocator& Allocator)
+inline void ParseRSN(const nlohmann::json& Json, ShaderMacroArray& Macros, DynamicLinearAllocator& Allocator)
 {
     if (!Json.is_array())
         throw nlohmann::json::type_error::create(JsonTypeError, std::string("type must be array, but is ") + Json.type_name(), Json);
 
-    auto* pData = Allocator.ConstructArray<ShaderMacro>(Json.size() + 1);
+    auto* pData = Allocator.ConstructArray<ShaderMacro>(Json.size());
     for (size_t i = 0; i < Json.size(); i++)
         ParseRSN(Json[i], pData[i], Allocator);
 
-    pMacros = pData;
+    Macros.Elements = pData;
+    Macros.Count    = static_cast<Uint32>(Json.size());
 }
 
 inline void ParseRSN(const nlohmann::json& Json, const Char**& pObjects, Uint32& NumElements, DynamicLinearAllocator& Allocator)

diff --git a/Tests/DiligentToolsTest/src/RenderStateNotationParser/ShaderParserTest.cpp b/Tests/DiligentToolsTest/src/RenderStateNotationParser/ShaderParserTest.cpp
@@ -122,9 +122,9 @@ TEST(Tools_RenderStateNotationParser, ParseShaderCreateInfo)
     EXPECT_STREQ(DescReference.Desc.Name, Desc.Desc.Name);
     EXPECT_EQ(DescReference.Desc, Desc.Desc);
     EXPECT_EQ(DescReference.SourceLanguage, Desc.SourceLanguage);
+    EXPECT_EQ(DescReference.Macros.Count, Desc.Macros.Count);
     EXPECT_EQ(DescReference.Macros[0], Desc.Macros[0]);
     EXPECT_EQ(DescReference.Macros[1], Desc.Macros[1]);
-    EXPECT_EQ(DescReference.Macros[2], Desc.Macros[2]);
 
     EXPECT_TRUE(SafeStrEqual(DescReference.FilePath, Desc.FilePath));
     EXPECT_TRUE(SafeStrEqual(DescReference.EntryPoint, Desc.EntryPoint));