sc/source/ui/view/tabvwsha.cxx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
New commits:
commit 95706c680c509016b8cd60c026d820c7d4575a12
Author: Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Thu Jul 14 12:45:07 2022 +0200
Commit: Stephan Bergmann <sberg...@redhat.com>
CommitDate: Thu Jul 14 16:18:21 2022 +0200
Drop some needless casts
...that were present ever since the surrounding code was added in
95b27dd2e5b5bdcb39962f72882dcdf406bcb393 "tdf#45705 rework zoom in and zoom
out
UI commands in Calc", but for no apparent reason (when MAXZOOM and MINZOOM
were
macros expanding to unadorned int literals). But even after
ac2a6ee9618e377806e529ed641f67e88684f7e7 "basegfx: zoomIn() and zoomOut()
should
be sal_uInt16" changed MAXZOOM and MINZOOM to be constants of type
sal_uInt16,
contemporary compilers are unlikely to emit warnings for these nominally
signed-vs.-unsigned comparisons.
Change-Id: If4481a2e539632dbdca2ce4b13aab63fb84225b7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/137075
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sberg...@redhat.com>
diff --git a/sc/source/ui/view/tabvwsha.cxx b/sc/source/ui/view/tabvwsha.cxx
index 67cd905341f0..dc389eb70723 100644
--- a/sc/source/ui/view/tabvwsha.cxx
+++ b/sc/source/ui/view/tabvwsha.cxx
@@ -306,7 +306,7 @@ void ScTabViewShell::GetState( SfxItemSet& rSet )
{
const Fraction& rZoomY = GetViewData().GetZoomY();
tools::Long nZoom = tools::Long(rZoomY * 100);
- if (nZoom >= tools::Long(MAXZOOM))
+ if (nZoom >= MAXZOOM)
rSet.DisableItem(nWhich);
}
break;
@@ -314,7 +314,7 @@ void ScTabViewShell::GetState( SfxItemSet& rSet )
{
const Fraction& rZoomY = GetViewData().GetZoomY();
tools::Long nZoom = tools::Long(rZoomY * 100);
- if (nZoom <= tools::Long(MINZOOM))
+ if (nZoom <= MINZOOM)
rSet.DisableItem(nWhich);
}
break;
