You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by le...@apache.org on 2012/11/01 19:05:52 UTC
svn commit: r1404698 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java
Author: lehmi
Date: Thu Nov 1 18:05:52 2012
New Revision: 1404698
URL: http://svn.apache.org/viewvc?rev=1404698&view=rev
Log:
PDFBOX-1362: drop backslash if it isn't used within an escape sequence as proposed by Christian Czech
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java?rev=1404698&r1=1404697&r2=1404698&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdfparser/BaseParser.java Thu Nov 1 18:05:52 2012
@@ -925,12 +925,9 @@ public abstract class BaseParser
}
default:
{
- retval.append( '\\' );
+ // dropping the backslash
+ // see 7.3.4.2 Literal Strings for further information
retval.append( next );
- //another problem with PDF's, sometimes the \ doesn't really
- //mean escape like the PDF spec says it does, sometimes is should be literal
- //which is what we will assume here.
- //throw new IOException( "Unexpected break sequence '" + next + "' " + pdfSource );
}
}
}