[mapserver-commits] r9915 - in trunk/mapserver: . mapscript/swiginc

[svn at osgeo.org]

Author: aboudreault
Date: 2010-03-05 09:47:03 -0500 (Fri, 05 Mar 2010)
New Revision: 9915

Modified:
   trunk/mapserver/HISTORY.TXT
   trunk/mapserver/mapscript/swiginc/mapzoom.i
Log:
Fixed swig zoomRectangle() method: the maxy in the rect object have to be < miny value (#3286)

Modified: trunk/mapserver/HISTORY.TXT
===================================================================
--- trunk/mapserver/HISTORY.TXT	2010-03-03 18:07:09 UTC (rev 9914)
+++ trunk/mapserver/HISTORY.TXT	2010-03-05 14:47:03 UTC (rev 9915)
@@ -14,6 +14,8 @@
 Current Version (SVN trunk):
 ----------------------------
 
+- Fixed swig zoomRectangle() method: the maxy in the rect object have to be < miny value (#3286)
+
 - Fix crash with GRID layers with no classes (#3352)
 
 - Remove "legacy" raster support, all raster rendering via GDAL now.

Modified: trunk/mapserver/mapscript/swiginc/mapzoom.i
===================================================================
--- trunk/mapserver/mapscript/swiginc/mapzoom.i	2010-03-03 18:07:09 UTC (rev 9914)
+++ trunk/mapserver/mapscript/swiginc/mapzoom.i	2010-03-05 14:47:03 UTC (rev 9915)
@@ -226,7 +226,9 @@
         /* ----------------------------------------------------------- */
         /*      check if the values passed are consistent min <= max.   */
         /* ----------------------------------------------------------- */
-        if (poPixRect->minx >= poPixRect->maxx) {
+        /* This is not a typo: "maxx >= minx". For historical reason, we
+         * keep this as it is. See documentation for more info about this check. */
+        if (poPixRect->maxx >= poPixRect->minx) {
             msSetError(MS_MISCERR, "image rectangle minx >= maxx", "mapscript::mapObj::zoomRectangle()");
             return MS_FAILURE;
         }