svgio/inc/svgtoken.hxx | 2
svgio/source/svgreader/svgtoken.cxx | 206 ++----------------------------------
2 files changed, 18 insertions(+), 190 deletions(-)
New commits:
commit 009ea6f786e5a7fd1e08d768e33b5b7af30e5765
Author: Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Fri Jul 7 12:46:24 2023 +0300
Commit: Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Fri Oct 27 06:35:26 2023 +0200
Commit 04c78e1a46a423071d7ea68724525ec7ef92e0e8 follow-up
Drops duplicating map, and uses std::find_if with case-insensitive
comparison. Likely to have some performance impact, but avoids the
maintenance penalty of two maps.
Change-Id: I7144556488aa134d9b9214b00ba9d7c4e0fb68a3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154159
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>
diff --git a/svgio/inc/svgtoken.hxx b/svgio/inc/svgtoken.hxx
index 3a4a89f285e2..3927a27d1db5 100644
--- a/svgio/inc/svgtoken.hxx
+++ b/svgio/inc/svgtoken.hxx
@@ -192,7 +192,7 @@ namespace svgio::svgreader
DominantBaseline
};
- SVGToken StrToSVGToken(const OUString& rStr, bool bCaseIndependent);
+ SVGToken StrToSVGToken(std::u16string_view rStr, bool bIgnoreCase);
OUString SVGTokenToStr(const SVGToken& rToken);
} // end of namespace svgio::svgreader
diff --git a/svgio/source/svgreader/svgtoken.cxx
b/svgio/source/svgreader/svgtoken.cxx
index 492c78623f14..fa28c8647cea 100644
--- a/svgio/source/svgreader/svgtoken.cxx
+++ b/svgio/source/svgreader/svgtoken.cxx
@@ -21,11 +21,12 @@
#include <frozen/bits/defines.h>
#include <frozen/bits/elsa_std.h>
#include <frozen/unordered_map.h>
+#include <o3tl/string_view.hxx>
namespace svgio::svgreader
{
-constexpr auto aSVGTokenMapperList =
frozen::make_unordered_map<std::u16string_view, SVGToken>(
+constexpr auto aSVGTokenMap = frozen::make_unordered_map<std::u16string_view,
SVGToken>(
{
{ u"width", SVGToken::Width },
{ u"height", SVGToken::Height },
@@ -181,203 +182,30 @@ constexpr auto aSVGTokenMapperList =
frozen::make_unordered_map<std::u16string_v
{ u"dominant-baseline", SVGToken::DominantBaseline }
});
-// The same elements as the map above but lowercase. CSS is case insensitive
-// TODO: create separate maps for css and xml elements
-constexpr auto aSVGLowerCaseTokenMapperList =
frozen::make_unordered_map<std::u16string_view, SVGToken>(
+SVGToken StrToSVGToken(std::u16string_view rStr, bool bIgnoreCase)
{
- { u"width", SVGToken::Width },
- { u"height", SVGToken::Height },
- { u"viewbox", SVGToken::ViewBox },
- { u"transform", SVGToken::Transform },
- { u"style", SVGToken::Style },
- { u"display", SVGToken::Display }, // #i121656#
- { u"d", SVGToken::D },
- { u"x", SVGToken::X },
- { u"y", SVGToken::Y },
- { u"xmlns", SVGToken::Xmlns },
- { u"version", SVGToken::Version },
- { u"id", SVGToken::Id },
- { u"in", SVGToken::In },
- { u"rx", SVGToken::Rx },
- { u"ry", SVGToken::Ry },
- { u"points", SVGToken::Points },
- { u"dx", SVGToken::Dx },
- { u"dy", SVGToken::Dy },
- { u"rotate", SVGToken::Rotate },
- { u"textlength", SVGToken::TextLength },
- { u"lengthadjust", SVGToken::LengthAdjust },
- { u"font", SVGToken::Font },
- { u"font-family", SVGToken::FontFamily },
- { u"font-size", SVGToken::FontSize },
- { u"font-size-adjust", SVGToken::FontSizeAdjust },
- { u"font-stretch", SVGToken::FontStretch },
- { u"font-style", SVGToken::FontStyle },
- { u"font-variant", SVGToken::FontVariant },
- { u"font-weight", SVGToken::FontWeight },
- { u"direction", SVGToken::Direction },
- { u"letter-spacing", SVGToken::LetterSpacing },
- { u"text-decoration", SVGToken::TextDecoration },
- { u"unicode-bidi", SVGToken::UnicodeBidi },
- { u"word-spacing", SVGToken::WordSpacing },
- { u"tspan", SVGToken::Tspan },
- { u"tref", SVGToken::Tref },
- { u"textpath", SVGToken::TextPath },
- { u"startoffset", SVGToken::StartOffset },
- { u"method", SVGToken::Method },
- { u"spacing", SVGToken::Spacing },
- { u"stddeviation", SVGToken::StdDeviation },
- { u"text-align", SVGToken::TextAlign },
- { u"pathlength", SVGToken::PathLength },
- { u"type", SVGToken::Type },
- { u"class", SVGToken::Class },
- { u"text-anchor", SVGToken::TextAnchor },
- { u"xml:space", SVGToken::XmlSpace },
- { u"color", SVGToken::Color },
- { u"clippath", SVGToken::ClipPathNode },
- { u"clip-path", SVGToken::ClipPathProperty },
- { u"fecolormatrix", SVGToken::FeColorMatrix },
- { u"fedropshadow", SVGToken::FeDropShadow },
- { u"feflood", SVGToken::FeFlood },
- { u"feimage", SVGToken::FeImage },
- { u"fegaussianblur", SVGToken::FeGaussianBlur },
- { u"feoffset", SVGToken::FeOffset },
- { u"filter", SVGToken::Filter },
- { u"flood-color", SVGToken::FloodColor },
- { u"flood-opacity", SVGToken::FloodOpacity },
- { u"mask", SVGToken::Mask },
- { u"clippathunits", SVGToken::ClipPathUnits },
- { u"maskunits", SVGToken::MaskUnits },
- { u"maskcontentunits", SVGToken::MaskContentUnits },
- { u"clip-rule", SVGToken::ClipRule },
- { u"marker", SVGToken::Marker },
- { u"marker-start", SVGToken::MarkerStart },
- { u"marker-mid", SVGToken::MarkerMid },
- { u"marker-end", SVGToken::MarkerEnd },
- { u"refx", SVGToken::RefX },
- { u"refy", SVGToken::RefY },
- { u"markerunits", SVGToken::MarkerUnits },
- { u"markerwidth", SVGToken::MarkerWidth },
- { u"markerheight", SVGToken::MarkerHeight },
- { u"orient", SVGToken::Orient },
- { u"pattern", SVGToken::Pattern },
- { u"patternunits", SVGToken::PatternUnits },
- { u"patterncontentunits", SVGToken::PatternContentUnits },
- { u"patterntransform", SVGToken::PatternTransform },
- { u"opacity", SVGToken::Opacity },
- { u"visibility", SVGToken::Visibility },
- { u"title", SVGToken::Title },
- { u"desc", SVGToken::Desc },
- { u"preserveaspectratio", SVGToken::PreserveAspectRatio },
- { u"defer", SVGToken::Defer },
- { u"none", SVGToken::None },
- { u"xminymin", SVGToken::XMinYMin },
- { u"xmidymin", SVGToken::XMidYMin },
- { u"xmaxymin", SVGToken::XMaxYMin },
- { u"xminymid", SVGToken::XMinYMid },
- { u"xmidymid", SVGToken::XMidYMid },
- { u"xmaxymid", SVGToken::XMaxYMid },
- { u"xminymax", SVGToken::XMinYMax },
- { u"xmidymax", SVGToken::XMidYMax },
- { u"xmaxymax", SVGToken::XMaxYMax },
- { u"meet", SVGToken::Meet },
- { u"slice", SVGToken::Slice },
- { u"values", SVGToken::Values },
+ std::u16string_view aStr = rStr.starts_with(u"svg:") ? rStr.substr(4) :
rStr;
- { u"defs", SVGToken::Defs },
- { u"g", SVGToken::G },
- { u"svg", SVGToken::Svg },
- { u"symbol", SVGToken::Symbol },
- { u"switch", SVGToken::Switch },
- { u"use", SVGToken::Use },
- { u"a", SVGToken::A },
+ // TODO: a better alternative to the bIgnoreCase would be separate maps
for SVG and CSS,
+ // the latter using case-insensitive hasher and comparator, with separate
search functions.
- { u"circle", SVGToken::Circle },
- { u"ellipse", SVGToken::Ellipse },
- { u"line", SVGToken::Line },
- { u"path", SVGToken::Path },
- { u"polygon", SVGToken::Polygon },
- { u"polyline", SVGToken::Polyline },
- { u"rect", SVGToken::Rect },
- { u"image", SVGToken::Image },
-
- { u"lineargradient", SVGToken::LinearGradient },
- { u"radialgradient", SVGToken::RadialGradient },
- { u"stop", SVGToken::Stop },
- { u"offset", SVGToken::Offset },
- { u"x1", SVGToken::X1 },
- { u"y1", SVGToken::Y1 },
- { u"x2", SVGToken::X2 },
- { u"y2", SVGToken::Y2 },
- { u"cx", SVGToken::Cx },
- { u"cy", SVGToken::Cy },
- { u"fx", SVGToken::Fx },
- { u"fy", SVGToken::Fy },
- { u"r", SVGToken::R },
- { u"gradientunits", SVGToken::GradientUnits },
- { u"gradienttransform", SVGToken::GradientTransform },
- { u"spreadmethod", SVGToken::SpreadMethod },
- { u"href", SVGToken::Href },
- { u"xlink:href", SVGToken::XlinkHref },
- { u"stop-color", SVGToken::StopColor },
- { u"stop-opacity", SVGToken::StopOpacity },
-
- { u"fill", SVGToken::Fill },
- { u"fill-opacity", SVGToken::FillOpacity },
- { u"fill-rule", SVGToken::FillRule },
-
- { u"stroke", SVGToken::Stroke },
- { u"stroke-dasharray", SVGToken::StrokeDasharray },
- { u"stroke-dashoffset", SVGToken::StrokeDashoffset },
- { u"stroke-linecap", SVGToken::StrokeLinecap },
- { u"stroke-linejoin", SVGToken::StrokeLinejoin },
- { u"stroke-miterlimit", SVGToken::StrokeMiterlimit },
- { u"stroke-opacity", SVGToken::StrokeOpacity },
- { u"stroke-width", SVGToken::StrokeWidth },
-
- { u"text", SVGToken::Text },
- { u"baseline-shift", SVGToken::BaselineShift },
- { u"dominant-baseline", SVGToken::DominantBaseline }
-});
-
-static_assert(sizeof(aSVGTokenMapperList) ==
sizeof(aSVGLowerCaseTokenMapperList),
- "Number of elements in both maps must be the same");
-
-SVGToken StrToSVGToken(const OUString& rStr, bool bCaseIndependent)
-{
- OUString aSearchString = rStr.startsWith("svg:") ? rStr.copy(4) : rStr;
-
- if (bCaseIndependent)
- {
- auto const
aResult(aSVGLowerCaseTokenMapperList.find(aSearchString.toAsciiLowerCase()));
-
- if (aResult != aSVGLowerCaseTokenMapperList.end())
- {
- return aResult->second;
- }
- }
- else
- {
- auto const aResult(aSVGTokenMapperList.find(aSearchString));
-
- if (aResult != aSVGTokenMapperList.end())
- {
- return aResult->second;
- }
- }
+ auto it = bIgnoreCase ? std::find_if(aSVGTokenMap.begin(),
aSVGTokenMap.end(),
+ [aStr](const auto& el)
+ { return
o3tl::equalsIgnoreAsciiCase(el.first, aStr); })
+ : aSVGTokenMap.find(aStr);
+ if (it != aSVGTokenMap.end())
+ return it->second;
return SVGToken::Unknown;
}
OUString SVGTokenToStr(const SVGToken& rToken)
{
- for (auto it = aSVGTokenMapperList.begin(); it !=
aSVGTokenMapperList.end(); ++it)
- {
- if (it->second == rToken)
- {
- OUString aFirst(it->first);
- return aFirst;
- }
- }
+ auto it = std::find_if(aSVGTokenMap.begin(), aSVGTokenMap.end(),
+ [rToken](const auto& el) { return el.second ==
rToken; });
+ if (it != aSVGTokenMap.end())
+ return OUString(it->first);
+
return OUString();
}
