AnimatedMRI: Don't try to scale factor 1

stanio · stanio · commit 4284e65a8d0c · 2025-05-11T15:35:37.000+03:00
Exception in thread "AWT-EventQueue-0" java.lang.IllegalArgumentException: Illegal scaling factor: 1
    	at io.github.stanio.xbrz.Scaler.forFactor(Scaler.java:29)
    	at io.github.stanio.xbrz.Xbrz.&lt;init&gt;(Xbrz.java:127)
    	at io.github.stanio.xbrz.awt.AnimatedMultiResolutionImage.lambda$0(AnimatedMultiResolutionImage.java:52)
    	at io.github.stanio.xbrz.awt.AnimatedMultiResolutionImage.getResolutionVariant(AnimatedMultiResolutionImage.java:51)
diff --git a/xbrz-awt/src/main/java9/io/github/stanio/xbrz/awt/AnimatedMultiResolutionImage.java b/xbrz-awt/src/main/java9/io/github/stanio/xbrz/awt/AnimatedMultiResolutionImage.java
@@ -48,9 +48,12 @@ public Image getResolutionVariant(double destWidth, double destHeight) {
             ((SuspendableFilteredSource) lastSource).stopProduction();
         }
 
+        lastFactor = factor;
+        if (factor == 1) {
+            return (lastVariant = baseImage);
+        }
         Xbrz scaler = scalers.computeIfAbsent(factor, k ->
                 new Xbrz(k, true, new ScalerCfg(), ColorDistance.bufferedYCbCr(5)));
-        lastFactor = factor;
         return lastVariant = preloadDimensions(
                 XbrzFilter.createScaledImage(baseImage, scaler));
     }

Original file line number	Diff line number	Diff line change
`@@ -48,9 +48,12 @@ public Image getResolutionVariant(double destWidth, double destHeight) {`
`48`	`48`	`((SuspendableFilteredSource) lastSource).stopProduction();`
`49`	`49`	`}`
`50`	`50`
	`51`	`+ lastFactor = factor;`
	`52`	`+ if (factor == 1) {`
	`53`	`+ return (lastVariant = baseImage);`
	`54`	`+ }`
`51`	`55`	`Xbrz scaler = scalers.computeIfAbsent(factor, k ->`
`52`	`56`	`new Xbrz(k, true, new ScalerCfg(), ColorDistance.bufferedYCbCr(5)));`
`53`		`- lastFactor = factor;`
`54`	`57`	`return lastVariant = preloadDimensions(`
`55`	`58`	`XbrzFilter.createScaledImage(baseImage, scaler));`
`56`	`59`	`}`