You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2022/01/21 18:57:10 UTC

svn commit: r1897310 - /pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/GlyfCompositeDescript.java

Author: tilman
Date: Fri Jan 21 18:57:10 2022
New Revision: 1897310

URL: http://svn.apache.org/viewvc?rev=1897310&view=rev
Log:
PDFBOX-4892: LGTM fix

Modified:
    pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/GlyfCompositeDescript.java

Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/GlyfCompositeDescript.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/GlyfCompositeDescript.java?rev=1897310&r1=1897309&r2=1897310&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/GlyfCompositeDescript.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/ttf/GlyfCompositeDescript.java Fri Jan 21 18:57:10 2022
@@ -160,9 +160,7 @@ public class GlyfCompositeDescript exten
             int n = i - c.getFirstIndex();
             int x = gd.getXCoordinate(n);
             int y = gd.getYCoordinate(n);
-            short x1 = (short) c.scaleX(x, y);
-            x1 += c.getXTranslate();
-            return x1;
+            return (short) (c.scaleX(x, y) + c.getXTranslate());
         }
         return 0;
     }
@@ -180,9 +178,7 @@ public class GlyfCompositeDescript exten
             int n = i - c.getFirstIndex();
             int x = gd.getXCoordinate(n);
             int y = gd.getYCoordinate(n);
-            short y1 = (short) c.scaleY(x, y);
-            y1 += c.getYTranslate();
-            return y1;
+            return (short) (c.scaleY(x, y) + c.getYTranslate());
         }
         return 0;
     }