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/10/16 11:04:06 UTC
svn commit: r1904609 - in /pdfbox/trunk: fontbox/src/main/java/org/apache/fontbox/cff/ pdfbox/src/main/java/org/apache/pdfbox/multipdf/ pdfbox/src/main/java/org/apache/pdfbox/pdmodel/ pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/ tools/src/main/j...
Author: tilman
Date: Sun Oct 16 11:04:05 2022
New Revision: 1904609
URL: http://svn.apache.org/viewvc?rev=1904609&view=rev
Log:
PDFBOX-4892: optimize, as suggested by valerybokov
Modified:
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1FontUtil.java
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDAbstractContentStream.java
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFField.java
pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFText2HTML.java
pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java
Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java?rev=1904609&r1=1904608&r2=1904609&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java Sun Oct 16 11:04:05 2022
@@ -403,7 +403,7 @@ public class CFFParser
exponentMissing = false;
break;
case 0xa:
- sb.append(".");
+ sb.append('.');
break;
case 0xb:
if (hasExponent)
@@ -411,7 +411,7 @@ public class CFFParser
LOG.warn("duplicate 'E' ignored after " + sb);
break;
}
- sb.append("E");
+ sb.append('E');
exponentMissing = true;
hasExponent = true;
break;
@@ -428,7 +428,7 @@ public class CFFParser
case 0xd:
break;
case 0xe:
- sb.append("-");
+ sb.append('-');
break;
case 0xf:
done = true;
@@ -444,7 +444,7 @@ public class CFFParser
// the exponent is missing, just append "0" to avoid an exception
// not sure if 0 is the correct value, but it seems to fit
// see PDFBOX-1522
- sb.append("0");
+ sb.append('0');
}
if (sb.length() == 0)
{
Modified: pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1FontUtil.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1FontUtil.java?rev=1904609&r1=1904608&r2=1904609&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1FontUtil.java (original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/Type1FontUtil.java Sun Oct 16 11:04:05 2022
@@ -43,7 +43,7 @@ public final class Type1FontUtil
String string = Integer.toHexString(aByte & 0xff);
if (string.length() == 1)
{
- sb.append("0");
+ sb.append('0');
}
sb.append(string.toUpperCase(Locale.US));
}
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java?rev=1904609&r1=1904608&r2=1904609&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/multipdf/Overlay.java Sun Oct 16 11:04:05 2022
@@ -550,7 +550,7 @@ public class Overlay implements Closeabl
for (double v : flatmatrix)
{
overlayStream.append(float2String((float) v));
- overlayStream.append(" ");
+ overlayStream.append(' ');
}
overlayStream.append(" cm\n");
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDAbstractContentStream.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDAbstractContentStream.java?rev=1904609&r1=1904608&r2=1904609&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDAbstractContentStream.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/PDAbstractContentStream.java Sun Oct 16 11:04:05 2022
@@ -504,20 +504,20 @@ abstract class PDAbstractContentStream i
sb.append(inlineImage.getHeight());
sb.append("\n /CS ");
- sb.append("/");
+ sb.append('/');
sb.append(inlineImage.getColorSpace().getName());
COSArray decodeArray = inlineImage.getDecode();
if (decodeArray != null && decodeArray.size() > 0)
{
sb.append("\n /D ");
- sb.append("[");
+ sb.append('[');
for (COSBase base : decodeArray)
{
sb.append(((COSNumber) base).intValue());
- sb.append(" ");
+ sb.append(' ');
}
- sb.append("]");
+ sb.append(']');
}
if (inlineImage.isStencil())
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFField.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFField.java?rev=1904609&r1=1904608&r2=1904609&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFField.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/fdf/FDFField.java Sun Oct 16 11:04:05 2022
@@ -791,7 +791,7 @@ public class FDFField implements COSObje
default:
if (c > 0x7e)
{
- escapedXML.append("&#").append((int) c).append(";");
+ escapedXML.append("&#").append((int) c).append(';');
}
else
{
Modified: pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFText2HTML.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFText2HTML.java?rev=1904609&r1=1904608&r2=1904609&view=diff
==============================================================================
--- pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFText2HTML.java (original)
+++ pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/PDFText2HTML.java Sun Oct 16 11:04:05 2022
@@ -220,7 +220,7 @@ public class PDFText2HTML extends PDFTex
// write non-ASCII as named entities
if ((character < 32) || (character > 126))
{
- builder.append("&#").append((int) character).append(";");
+ builder.append("&#").append((int) character).append(';');
}
else
{
Modified: pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java?rev=1904609&r1=1904608&r2=1904609&view=diff
==============================================================================
--- pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java (original)
+++ pdfbox/trunk/tools/src/main/java/org/apache/pdfbox/tools/TextToPDF.java Sun Oct 16 11:04:05 2022
@@ -255,7 +255,7 @@ public class TextToPDF implements Callab
{
if (addSpace)
{
- nextLineToDraw.append(" ");
+ nextLineToDraw.append(' ');
}
else
{