[Libreoffice-commits] core.git: cui/source

Rosen rosen at live.com
Sun Sep 25 07:16:36 UTC 2016


 cui/source/dialogs/colorpicker.cxx |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

New commits:
commit dad35b0463a16d3d6eca068100e95822ae466a9a
Author: Rosen <rosen at live.com>
Date:   Sat Sep 24 11:11:27 2016 +0530

    tdf#96505  Get rid of cargo cult "long" integer literals
    
    Change-Id: Ic1602a57d1662cb30b2339de9eeb6483fdfbef37
    Reviewed-on: https://gerrit.libreoffice.org/29242
    Reviewed-by: jan iversen <jani at documentfoundation.org>
    Tested-by: jan iversen <jani at documentfoundation.org>

diff --git a/cui/source/dialogs/colorpicker.cxx b/cui/source/dialogs/colorpicker.cxx
index c9161b7..05111e1 100644
--- a/cui/source/dialogs/colorpicker.cxx
+++ b/cui/source/dialogs/colorpicker.cxx
@@ -441,15 +441,15 @@ void ColorFieldControl::ShowPosition( const Point& rPos, bool bUpdate )
 
     long nX = rPos.X();
     long nY = rPos.Y();
-    if (nX < 0L)
-        nX = 0L;
+    if (nX < 0)
+        nX = 0;
     else if (nX >= aSize.Width())
-        nX = aSize.Width() - 1L;
+        nX = aSize.Width() - 1;
 
-    if (nY < 0L)
-        nY = 0L;
+    if (nY < 0)
+        nY = 0;
     else if (nY >= aSize.Height())
-        nY = aSize.Height() - 1L;
+        nY = aSize.Height() - 1;
 
     Point aPos = maPosition;
     maPosition.X() = nX - 5;
@@ -765,7 +765,7 @@ void ColorSliderControl::ChangePosition(long nY)
 {
     const long nHeight = GetOutputSizePixel().Height() - 1;
 
-    if (nY < 0L)
+    if (nY < 0)
         nY = 0;
     else if (nY > nHeight)
         nY = nHeight;


More information about the Libreoffice-commits mailing list