Add check when removing the baseId prefix

CMake/ctre.cmake

@@ -1,3 +1,4 @@
+if(NOT TARGET pugixml)
     include(FetchContent)
 
     FetchContent_Declare(
@@ -9,3 +10,4 @@ FetchContent_Declare(
     FetchContent_MakeAvailable(
         ctre
     )
+endif()

CMake/pugixml.cmake

@@ -1,3 +1,4 @@
+if(NOT TARGET pugixml)
     include(FetchContent)
 
     FetchContent_Declare(
@@ -9,3 +10,4 @@ FetchContent_Declare(
     FetchContent_MakeAvailable(
         pugixml
     )
+endif()

Libraries/Kanimaji/Source/Kanimaji.cpp

@@ -103,12 +103,14 @@ namespace Kanimaji
                 return std::string_view(node.attribute("id").as_string()).contains("StrokeNumbers");
             }));
 
-            pugi::xml_node pathsContainer = svg.find_child([](pugi::xml_node& node) {
+            pugi::xml_node pathsGroup = svg.find_child([](pugi::xml_node& node) {
                 return std::string_view(node.attribute("id").as_string()).contains("StrokePaths");
             });
-            pathsContainer.attribute("style") = "fill:none;visibility:hidden;";
+            pathsGroup.attribute("style") = "fill:none;visibility:hidden;";
-            std::string baseId = pathsContainer.attribute("id").as_string();
+            std::string baseId = pathsGroup.attribute("id").as_string();
-            baseId.erase(0, baseId.find('_') + 1);
+            if (auto pos = baseId.find('_'); pos != baseId.npos) {
+                baseId.erase(0, pos + 1);
+            }
 
             // 1st pass for getting information
             double totalLength = 0.0;
@@ -124,9 +126,9 @@ namespace Kanimaji
             std::string brushKeyframes;
             std::string brushProgress;
 
-            auto background = AppendGroup(svg, "kvg:Kanimaji_Stroke_Background_" + baseId,
+            auto background = AppendGroup(svg, "kvg:Kanimaji_StrokeBackground_" + baseId,
                                           settings.UnfilledStroke);
-            auto brushBorder = AppendGroup(svg, "kvg:Kanimaji_Brush_Border_" + baseId,
+            auto brushBorder = AppendGroup(svg, "kvg:Kanimaji_BrushBorder_" + baseId,
                                            settings.BrushBorder);
             auto animation = AppendGroup(svg, "kvg:Kanimaji_Animation_" + baseId,
                                          settings.FilledStroke);
@@ -138,6 +140,7 @@ namespace Kanimaji
                              - AsSeconds(settings.DelayBetweenStrokes);
             double previousLength = 0.0;
             double currentLength = 0.0;
+
             // 2nd pass to prepare the CSS
             for (const auto& path : paths) {
                 std::string_view d = path.node().attribute("d").as_string();
@@ -158,8 +161,8 @@ namespace Kanimaji
 
                 std::string id = idAttr.as_string();
                 std::string css = id;
-                if (auto colonPos = css.find(':'); colonPos != css.npos) {
+                if (auto pos = css.find(':'); pos != css.npos) {
-                    css.replace(css.find(":"), 1, "\\:");
+                    css.replace(pos, 1, "\\:");
                 }
 
                 std::string pathId = id;
@@ -217,7 +220,7 @@ namespace Kanimaji
             styles.append(brushProgress);
             styles.append("    ");
             pugi::xml_node style = svg.prepend_child("style");
-            style.append_attribute("id") = "kvg:Kanimaji_Style";
+            style.append_attribute("id") = "kvg:Kanimaji_Styles_" + baseId;
             style.append_child(pugi::node_pcdata).set_value(styles);
         }
     }

Tools/KanimajiTool/Main.cpp

@@ -16,7 +16,7 @@ int main(const int argc, const char *argv[])
         Kanimaji::AnimateFile(argv[1], argv[2]);
     }
     catch (const std::exception& e) {
-        std::println("Could not animate {}: {}", argv[1], argv[2]);
+        std::println("Could not animate {}: {}", argv[1], e.what());
         return 1;
     }