basegfx/source/color/bcolormodifier.cxx |   24 ++++++++++++------------
 basegfx/source/tools/bgradient.cxx      |   10 +++++-----
 basegfx/source/tools/canvastools.cxx    |    4 ++--
 basegfx/source/tools/unopolypolygon.cxx |    4 ++--
 download.lst                            |    4 ++--
 external/glm/UnpackedTarball_glm.mk     |    1 -
 external/glm/c++20.patch.0              |   11 -----------
 external/glm/clang-cl.patch.0           |   18 +++++++++---------
 8 files changed, 32 insertions(+), 44 deletions(-)

New commits:
commit 4ee04612875b30d122849038a00382356def6bc1
Author:     Noel Grandin <noel.gran...@collabora.co.uk>
AuthorDate: Mon May 6 10:17:42 2024 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Tue May 7 12:54:20 2024 +0200

    loplugin:ostr basegfx
    
    Change-Id: I79212cbfd1c7cae810f8e24dbd73372b9f97f9ef
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167190
    Tested-by: Noel Grandin <noel.gran...@collabora.co.uk>
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>

diff --git a/basegfx/source/color/bcolormodifier.cxx 
b/basegfx/source/color/bcolormodifier.cxx
index 3b9d1ce158c3..04ab3757cb45 100644
--- a/basegfx/source/color/bcolormodifier.cxx
+++ b/basegfx/source/color/bcolormodifier.cxx
@@ -47,7 +47,7 @@ namespace basegfx
 
     OUString BColorModifier_gray::getModifierName() const
     {
-        return "gray";
+        return u"gray"_ustr;
     }
 
     BColorModifier_invert::~BColorModifier_invert()
@@ -66,7 +66,7 @@ namespace basegfx
 
     OUString BColorModifier_invert::getModifierName() const
     {
-        return "invert";
+        return u"invert"_ustr;
     }
 
     BColorModifier_luminance_to_alpha::~BColorModifier_luminance_to_alpha()
@@ -87,7 +87,7 @@ namespace basegfx
 
     OUString BColorModifier_luminance_to_alpha::getModifierName() const
     {
-        return "luminance_to_alpha";
+        return u"luminance_to_alpha"_ustr;
     }
 
     BColorModifier_replace::~BColorModifier_replace()
@@ -113,7 +113,7 @@ namespace basegfx
 
     OUString BColorModifier_replace::getModifierName() const
     {
-        return "replace";
+        return u"replace"_ustr;
     }
 
     BColorModifier_interpolate::~BColorModifier_interpolate()
@@ -139,7 +139,7 @@ namespace basegfx
 
     OUString BColorModifier_interpolate::getModifierName() const
     {
-        return "interpolate";
+        return u"interpolate"_ustr;
     }
 
     BColorModifier_matrix::~BColorModifier_matrix()
@@ -194,7 +194,7 @@ namespace basegfx
 
     OUString BColorModifier_matrix::getModifierName() const
     {
-        return "matrix";
+        return u"matrix"_ustr;
     }
 
     BColorModifier_saturate::BColorModifier_saturate(double fValue)
@@ -239,7 +239,7 @@ namespace basegfx
 
     OUString BColorModifier_saturate::getModifierName() const
     {
-        return "saturate";
+        return u"saturate"_ustr;
     }
 
     BColorModifier_hueRotate::BColorModifier_hueRotate(double fRad)
@@ -290,7 +290,7 @@ namespace basegfx
 
     OUString BColorModifier_hueRotate::getModifierName() const
     {
-        return "hueRotate";
+        return u"hueRotate"_ustr;
     }
 
     BColorModifier_black_and_white::~BColorModifier_black_and_white()
@@ -325,7 +325,7 @@ namespace basegfx
 
     OUString BColorModifier_black_and_white::getModifierName() const
     {
-        return "black_and_white";
+        return u"black_and_white"_ustr;
     }
 
     BColorModifier_gamma::BColorModifier_gamma(double fValue)
@@ -376,7 +376,7 @@ namespace basegfx
 
     OUString BColorModifier_gamma::getModifierName() const
     {
-        return "gamma";
+        return u"gamma"_ustr;
     }
 
     
BColorModifier_RGBLuminanceContrast::BColorModifier_RGBLuminanceContrast(double 
fRed, double fGreen, double fBlue, double fLuminance, double fContrast)
@@ -458,7 +458,7 @@ namespace basegfx
 
     OUString BColorModifier_RGBLuminanceContrast::getModifierName() const
     {
-        return "RGBLuminanceContrast";
+        return u"RGBLuminanceContrast"_ustr;
     }
 
     BColorModifier_randomize::BColorModifier_randomize(double fRandomPart)
@@ -514,7 +514,7 @@ namespace basegfx
 
     OUString BColorModifier_randomize::getModifierName() const
     {
-        return "randomize";
+        return u"randomize"_ustr;
     }
 
     ::basegfx::BColor BColorModifierStack::getModifiedColor(const 
::basegfx::BColor& rSource) const
diff --git a/basegfx/source/tools/bgradient.cxx 
b/basegfx/source/tools/bgradient.cxx
index 9f98c54cfc1c..a8bbe95c526b 100644
--- a/basegfx/source/tools/bgradient.cxx
+++ b/basegfx/source/tools/bgradient.cxx
@@ -55,12 +55,12 @@ StringMap lcl_jsonToStringMap(std::u16string_view rJSON)
 
 basegfx::BGradient lcl_buildGradientFromStringMap(StringMap& rMap)
 {
-    basegfx::BGradient aGradient(
-        
basegfx::BColorStops(ColorToBColorConverter(rMap["startcolor"].toInt32(16)).getBColor(),
-                             
ColorToBColorConverter(rMap["endcolor"].toInt32(16)).getBColor()));
+    basegfx::BGradient aGradient(basegfx::BColorStops(
+        
ColorToBColorConverter(rMap[u"startcolor"_ustr].toInt32(16)).getBColor(),
+        
ColorToBColorConverter(rMap[u"endcolor"_ustr].toInt32(16)).getBColor()));
 
-    aGradient.SetGradientStyle(lcl_getStyleFromString(rMap["style"]));
-    aGradient.SetAngle(Degree10(rMap["angle"].toInt32()));
+    aGradient.SetGradientStyle(lcl_getStyleFromString(rMap[u"style"_ustr]));
+    aGradient.SetAngle(Degree10(rMap[u"angle"_ustr].toInt32()));
 
     return aGradient;
 }
diff --git a/basegfx/source/tools/canvastools.cxx 
b/basegfx/source/tools/canvastools.cxx
index 70f4787710ad..b46662419b95 100644
--- a/basegfx/source/tools/canvastools.cxx
+++ b/basegfx/source/tools/canvastools.cxx
@@ -287,8 +287,8 @@ namespace basegfx::unotools
                     if( !xLinePoly.is() )
                     {
                         throw lang::IllegalArgumentException(
-                            
"basegfx::unotools::b2DPolyPolygonFromXPolyPolygon2D(): Invalid input"
-                            "poly-polygon, cannot retrieve vertex data",
+                            
u"basegfx::unotools::b2DPolyPolygonFromXPolyPolygon2D(): Invalid input"
+                            "poly-polygon, cannot retrieve vertex data"_ustr,
                             uno::Reference< uno::XInterface >(),
                             0 );
                     }
diff --git a/basegfx/source/tools/unopolypolygon.cxx 
b/basegfx/source/tools/unopolypolygon.cxx
index 323e06a96f18..3f4d273cf8f2 100644
--- a/basegfx/source/tools/unopolypolygon.cxx
+++ b/basegfx/source/tools/unopolypolygon.cxx
@@ -95,8 +95,8 @@ namespace basegfx::unotools
                 // found - contract violation.
                 if( !xLinePoly.is() )
                     throw lang::IllegalArgumentException(
-                        "UnoPolyPolygon::addPolyPolygon(): Invalid input "
-                        "poly-polygon, cannot retrieve vertex data",
+                        u"UnoPolyPolygon::addPolyPolygon(): Invalid input "
+                        "poly-polygon, cannot retrieve vertex data"_ustr,
                         getXWeak(), 1);
 
                 aSrcPoly = unotools::polyPolygonFromPoint2DSequenceSequence(
commit 01fe29c78d21951be45bf8957092786d67c70b8e
Author:     Xisco Fauli <xiscofa...@libreoffice.org>
AuthorDate: Tue May 7 11:32:41 2024 +0200
Commit:     Xisco Fauli <xiscofa...@libreoffice.org>
CommitDate: Tue May 7 12:54:16 2024 +0200

    glm: Upgrade to 1.0.1
    
    c++20.patch.0 got obseleted in
    
https://github.com/g-truc/glm/commit/6fdeff4d67f3db493d47c44da20aa1efaa6574ef
    
    Change-Id: I8698968b55f299c55777fe280b3eda9e2bea9cdc
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/167268
    Tested-by: Jenkins
    Reviewed-by: Xisco Fauli <xiscofa...@libreoffice.org>

diff --git a/download.lst b/download.lst
index 28538879f5ec..27ffbbb97306 100644
--- a/download.lst
+++ b/download.lst
@@ -277,8 +277,8 @@ FROZEN_TARBALL := frozen-1.1.1.tar.gz
 # three static lines
 # so that git cherry-pick
 # will not run into conflicts
-GLM_SHA256SUM := 
6bba5f032bed47c73ad9397f2313b9acbfb56253d0d0576b5873d3dcb25e99ad
-GLM_TARBALL := glm-0.9.9.8.zip
+GLM_SHA256SUM := 
09c5716296787e1f7fcb87b1cbdbf26814ec1288ed6259ccd30d5d9795809fa5
+GLM_TARBALL := glm-1.0.1.zip
 # three static lines
 # so that git cherry-pick
 # will not run into conflicts
diff --git a/external/glm/UnpackedTarball_glm.mk 
b/external/glm/UnpackedTarball_glm.mk
index 934621eac841..78226f44cc08 100644
--- a/external/glm/UnpackedTarball_glm.mk
+++ b/external/glm/UnpackedTarball_glm.mk
@@ -17,7 +17,6 @@ $(eval $(call gb_UnpackedTarball_set_patchlevel,glm,1))
 
 $(eval $(call gb_UnpackedTarball_add_patches,glm, \
     external/glm/clang-cl.patch.0 \
-    external/glm/c++20.patch.0 \
 ))
 
 # vim: set noet sw=4 ts=4:
diff --git a/external/glm/c++20.patch.0 b/external/glm/c++20.patch.0
deleted file mode 100644
index 9fdd813a1e53..000000000000
--- a/external/glm/c++20.patch.0
+++ /dev/null
@@ -1,11 +0,0 @@
---- glm/detail/type_half.inl
-+++ glm/detail/type_half.inl
-@@ -6,7 +6,7 @@
-               volatile float f = 1e10;
- 
-               for(int i = 0; i < 10; ++i)
--                      f *= f; // this will overflow before the for loop 
terminates
-+                      f = f * f; // this will overflow before the for loop 
terminates
-               return f;
-       }
- 
diff --git a/external/glm/clang-cl.patch.0 b/external/glm/clang-cl.patch.0
index e5536e145094..6ba15fb788b1 100644
--- a/external/glm/clang-cl.patch.0
+++ b/external/glm/clang-cl.patch.0
@@ -3,12 +3,12 @@
 --- glm/detail/func_integer.inl
 +++ glm/detail/func_integer.inl
 @@ -3,7 +3,9 @@
- #include "_vectorize.hpp"
- #if(GLM_ARCH & GLM_ARCH_X86 && GLM_COMPILER & GLM_COMPILER_VC)
- #     include <intrin.h>
-+#if !defined __clang__
- #     pragma intrinsic(_BitScanReverse)
-+#endif
- #endif//(GLM_ARCH & GLM_ARCH_X86 && GLM_COMPILER & GLM_COMPILER_VC)
- #include <limits>
- 
+ #include "_vectorize.hpp"
+ #if(GLM_ARCH & GLM_ARCH_X86 && GLM_COMPILER & GLM_COMPILER_VC)
+ #     include <intrin.h>
++#if !defined __clang__
+ #     pragma intrinsic(_BitScanReverse)
++#endif
+ #endif//(GLM_ARCH & GLM_ARCH_X86 && GLM_COMPILER & GLM_COMPILER_VC)
+ #include <limits>
+ 

Reply via email to