diff --git a/CMake/ctre.cmake b/CMake/ctre.cmake index 6706f42..8cc43ce 100644 --- a/CMake/ctre.cmake +++ b/CMake/ctre.cmake @@ -1,13 +1,11 @@ -if(NOT TARGET pugixml) - include(FetchContent) +include(FetchContent) - FetchContent_Declare( - ctre - GIT_REPOSITORY https://code.3011.io/Mirrors/compile-time-regular-expressions - GIT_TAG v3.10.0 - ) +FetchContent_Declare( + ctre + GIT_REPOSITORY https://code.3011.io/Mirrors/compile-time-regular-expressions + GIT_TAG v3.10.0 +) - FetchContent_MakeAvailable( - ctre - ) -endif() +FetchContent_MakeAvailable( + ctre +) diff --git a/CMake/pugixml.cmake b/CMake/pugixml.cmake index fd5f750..f7df319 100644 --- a/CMake/pugixml.cmake +++ b/CMake/pugixml.cmake @@ -1,13 +1,11 @@ -if(NOT TARGET pugixml) - include(FetchContent) +include(FetchContent) - FetchContent_Declare( - pugixml - GIT_REPOSITORY https://code.3011.io/Mirrors/pugixml - GIT_TAG v1.15 - ) +FetchContent_Declare( + pugixml + GIT_REPOSITORY https://code.3011.io/Mirrors/pugixml + GIT_TAG v1.15 +) - FetchContent_MakeAvailable( - pugixml - ) -endif() +FetchContent_MakeAvailable( + pugixml +) diff --git a/Libraries/Kanimaji/Source/Kanimaji.cpp b/Libraries/Kanimaji/Source/Kanimaji.cpp index 27cdadd..7869e94 100644 --- a/Libraries/Kanimaji/Source/Kanimaji.cpp +++ b/Libraries/Kanimaji/Source/Kanimaji.cpp @@ -103,14 +103,12 @@ namespace Kanimaji return std::string_view(node.attribute("id").as_string()).contains("StrokeNumbers"); })); - pugi::xml_node pathsGroup = svg.find_child([](pugi::xml_node& node) { + pugi::xml_node pathsContainer = svg.find_child([](pugi::xml_node& node) { return std::string_view(node.attribute("id").as_string()).contains("StrokePaths"); }); - pathsGroup.attribute("style") = "fill:none;visibility:hidden;"; - std::string baseId = pathsGroup.attribute("id").as_string(); - if (auto pos = baseId.find('_'); pos != baseId.npos) { - baseId.erase(0, pos + 1); - } + pathsContainer.attribute("style") = "fill:none;visibility:hidden;"; + std::string baseId = pathsContainer.attribute("id").as_string(); + baseId.erase(0, baseId.find('_') + 1); // 1st pass for getting information double totalLength = 0.0; @@ -126,9 +124,9 @@ namespace Kanimaji std::string brushKeyframes; std::string brushProgress; - auto background = AppendGroup(svg, "kvg:Kanimaji_StrokeBackground_" + baseId, + auto background = AppendGroup(svg, "kvg:Kanimaji_Stroke_Background_" + baseId, settings.UnfilledStroke); - auto brushBorder = AppendGroup(svg, "kvg:Kanimaji_BrushBorder_" + baseId, + auto brushBorder = AppendGroup(svg, "kvg:Kanimaji_Brush_Border_" + baseId, settings.BrushBorder); auto animation = AppendGroup(svg, "kvg:Kanimaji_Animation_" + baseId, settings.FilledStroke); @@ -140,7 +138,6 @@ 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(); @@ -161,8 +158,8 @@ namespace Kanimaji std::string id = idAttr.as_string(); std::string css = id; - if (auto pos = css.find(':'); pos != css.npos) { - css.replace(pos, 1, "\\:"); + if (auto colonPos = css.find(':'); colonPos != css.npos) { + css.replace(css.find(":"), 1, "\\:"); } std::string pathId = id; @@ -220,7 +217,7 @@ namespace Kanimaji styles.append(brushProgress); styles.append(" "); pugi::xml_node style = svg.prepend_child("style"); - style.append_attribute("id") = "kvg:Kanimaji_Styles_" + baseId; + style.append_attribute("id") = "kvg:Kanimaji_Style"; style.append_child(pugi::node_pcdata).set_value(styles); } } diff --git a/Tools/KanimajiTool/Main.cpp b/Tools/KanimajiTool/Main.cpp index 383d717..ca7cb91 100644 --- a/Tools/KanimajiTool/Main.cpp +++ b/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], e.what()); + std::println("Could not animate {}: {}", argv[1], argv[2]); return 1; }