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 2017/07/28 15:33:22 UTC

svn commit: r1803285 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java

Author: tilman
Date: Fri Jul 28 15:33:21 2017
New Revision: 1803285

URL: http://svn.apache.org/viewvc?rev=1803285&view=rev
Log:
PDFBOX-3881: don't keep BOM for empty strings, as suggested by Nico Prenzel

Modified:
    pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java?rev=1803285&r1=1803284&r2=1803285&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSString.java Fri Jul 28 15:33:21 2017
@@ -185,7 +185,7 @@ public final class COSString extends COS
     public String getString()
     {
         // text string - BOM indicates Unicode
-        if (bytes.length > 2)
+        if (bytes.length >= 2)
         {
             if ((bytes[0] & 0xff) == 0xFE && (bytes[1] & 0xff) == 0xFF)
             {