svtools/source/misc/unitconv.cxx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
New commits:
commit d55b2631342bc7babf3709f2f83e0e502ebe4014
Author: Mike Kaganski <mike.kagan...@collabora.com>
AuthorDate: Tue Nov 9 09:22:43 2021 +0300
Commit: Mike Kaganski <mike.kagan...@collabora.com>
CommitDate: Tue Nov 9 08:43:03 2021 +0100
tdf#145158: pre-multiply the value by factor before conversion
Otherwise it is rounded before multiplying. Prior to the regressing
commit cfff893b9c82843a90aac4ecdb3a3936721b74a0, it was calculating
in twips, and was only converted to points by dividing by 20 in the
end, which allowed to keep some precision.
Change-Id: I142371dc630584f3fe64b5bfd5b592d46226ce32
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/124895
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kagan...@collabora.com>
diff --git a/svtools/source/misc/unitconv.cxx b/svtools/source/misc/unitconv.cxx
index bdff583182b5..5e316c5adf46 100644
--- a/svtools/source/misc/unitconv.cxx
+++ b/svtools/source/misc/unitconv.cxx
@@ -186,8 +186,8 @@ tools::Long CalcToPoint( tools::Long nIn, MapUnit eUnit,
sal_uInt16 nFactor )
eUnit == MapUnit::MapMM ||
eUnit == MapUnit::MapCM, "this unit is not implemented" );
- if (const auto eTo = MapToO3tlLength(eUnit); eTo != o3tl::Length::invalid)
- return o3tl::convert(nIn, eTo, o3tl::Length::pt) * nFactor;
+ if (const auto eFrom = MapToO3tlLength(eUnit); eFrom !=
o3tl::Length::invalid)
+ return o3tl::convert(nIn * static_cast<sal_Int64>(nFactor), eFrom,
o3tl::Length::pt);
return 0;
}
