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 2021/03/08 11:59:18 UTC
svn commit: r1887322 -
/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDDeviceN.java
Author: tilman
Date: Mon Mar 8 11:59:18 2021
New Revision: 1887322
URL: http://svn.apache.org/viewvc?rev=1887322&view=rev
Log:
PDFBOX-4060: use StringBuilder instead of String, as suggested by valerybokov
Modified:
pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDDeviceN.java
Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDDeviceN.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDDeviceN.java?rev=1887322&r1=1887321&r2=1887322&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDDeviceN.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/graphics/color/PDDeviceN.java Mon Mar 8 11:59:18 2021
@@ -283,6 +283,7 @@ public class PDDeviceN extends PDSpecial
// cache color mappings
Map<String, int[]> map1 = new HashMap<String, int[]>();
String key = null;
+ StringBuilder keyBuilder = new StringBuilder();
int width = raster.getWidth();
int height = raster.getHeight();
@@ -300,11 +301,13 @@ public class PDDeviceN extends PDSpecial
{
raster.getPixel(x, y, src);
// use a string representation as key
- key = Float.toString(src[0]);
+ keyBuilder.append(src[0]);
for (int s = 1; s < numSrcComponents; s++)
{
- key += "#" + Float.toString(src[s]);
+ keyBuilder.append('#').append(src[s]);
}
+ key = keyBuilder.toString();
+ keyBuilder.setLength(0);
int[] pxl = map1.get(key);
if (pxl != null)
{