[Libreoffice-commits] .: Branch 'libreoffice-3-4' - sc/source
Lionel Elie Mamane
lmamane at kemper.freedesktop.org
Thu Nov 24 02:13:43 PST 2011
sc/source/filter/excel/xeview.cxx | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
New commits:
commit 0bde24644b5457aa193ecc84ba37169a967de1c6
Author: Kohei Yoshida <kohei.yoshida at suse.com>
Date: Wed Nov 23 18:23:14 2011 -0500
fdo#39589: Avoid using ulimit_cast between signed and unsigned.
If you do, things go very very wrong. These *limit_cast try to be
clever and check ranges of various types before casting, but they
fail to do the right thing when casting a signed integer into unsigned.
Signed-off-by: Lionel Elie Mamane <lionel at mamane.lu>
diff --git a/sc/source/filter/excel/xeview.cxx b/sc/source/filter/excel/xeview.cxx
index 437a953..5a02c3f 100644
--- a/sc/source/filter/excel/xeview.cxx
+++ b/sc/source/filter/excel/xeview.cxx
@@ -368,8 +368,8 @@ XclExpTabViewSettings::XclExpTabViewSettings( const XclExpRoot& rRoot, SCTAB nSc
else
{
// split window: position is in twips
- maData.mnSplitX = ulimit_cast< sal_uInt16 >( rTabSett.maSplitPos.X() );
- maData.mnSplitY = ulimit_cast< sal_uInt32 >( rTabSett.maSplitPos.Y() );
+ maData.mnSplitX = static_cast<sal_uInt16>(rTabSett.maSplitPos.X());
+ maData.mnSplitY = static_cast<sal_uInt32>(rTabSett.maSplitPos.Y());
}
// selection
More information about the Libreoffice-commits
mailing list