Fixed Issue #65 (#65) and

added test-case to catch it in the future.

The stop-condition in incremental scaling (line 2266) was stopping too
early if either the width OR height didn't change between an incremental
scale.

Technically the incremental scaling shouldn't stop until BOTH dimensions
stop changing.

The fix was simple; change the check from || to &&.

Author: Riyad Kalla

src/main/java/org/imgscalr/Scalr.java

@@ -2251,9 +2251,20 @@ protected static BufferedImage scaleImageIncrementally(BufferedImage src,
 					currentHeight = targetHeight;
 			}
 
-			// Stop when we cannot incrementally step down anymore.
+			/*
+			 * Stop when we cannot incrementally step down anymore.
+			 * 
+			 * This used to use a || condition, but that would cause problems
+			 * when using FIT_EXACT such that sometimes the width OR height
+			 * would not change between iterations, but the other dimension
+			 * would (e.g. resizing 500x500 to 500x250).
+			 * 
+			 * Now changing this to an && condition requires that both
+			 * dimensions do not change between a resize iteration before we
+			 * consider ourselves done.
+			 */
 			if (prevCurrentWidth == currentWidth
-					|| prevCurrentHeight == currentHeight)
+					&& prevCurrentHeight == currentHeight)
 				break;
 
 			if (DEBUG)

src/test/java/org/imgscalr/ScalrResizeTest.java

@@ -93,6 +93,7 @@ public void testResizeWHSpeed() {
 
 	@Test
 	public void testResizeSizeExact() {
+		System.setProperty(Scalr.DEBUG_PROPERTY_NAME, "true");
 		assertEquals(load("time-square-resize-320-fit-exact.png"),
 				Scalr.resize(src, Mode.FIT_EXACT, 320));
 	}
@@ -134,4 +135,13 @@ public void testResizeUltraQuality() {
 		// out.
 		Assert.assertTrue(true);
 	}
+	
+	@Test
+	public void testResizeFitExact() {
+		BufferedImage i = new BufferedImage(500, 500, BufferedImage.TYPE_INT_RGB);
+		BufferedImage i2 = Scalr.resize(i, Mode.FIT_EXACT, 500, 250);
+		
+		Assert.assertEquals(i2.getWidth(), 500);
+		Assert.assertEquals(i2.getHeight(), 250);
+	}
 }