Author: msahyoun
Date: Tue Mar 17 12:29:19 2015
New Revision: 1667288
URL: http://svn.apache.org/r1667288
Log:
PDFBOX-2712 remove commented out lines
Modified:
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/FontMetrics.java
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cff/CFFParser.java
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java
Modified:
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/FontMetrics.java
URL:
http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/FontMetrics.java?rev=1667288&r1=1667287&r2=1667288&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/FontMetrics.java
(original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/afm/FontMetrics.java
Tue Mar 17 12:29:19 2015
@@ -94,8 +94,6 @@ public class FontMetrics
if( metric == null )
{
result=0;
- //don't throw an exception right away.
- //throw new IOException( "Unknown AFM(" + getFullName() + ")
characer '" + name + "'" );
}
else
{
@@ -117,8 +115,6 @@ public class FontMetrics
if( metric == null )
{
result=0;
- //don't throw an exception right away.
- //throw new IOException( "Unknown AFM(" + getFullName() + ")
characer '" + name + "'" );
}
else
{
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=1667288&r1=1667287&r2=1667288&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
Tue Mar 17 12:29:19 2015
@@ -598,9 +598,9 @@ public class CFFParser
input.setPosition(fdSelectPos);
FDSelect fdSelect = readFDSelect(input, charStringsIndex.getCount(),
font);
- // todo: almost certainly erroneous - CIDFonts do not have a top-level
private dict
- //font.addValueToPrivateDict("defaultWidthX", 1000);
- //font.addValueToPrivateDict("nominalWidthX", 0);
+ // TODO almost certainly erroneous - CIDFonts do not have a top-level
private dict
+ // font.addValueToPrivateDict("defaultWidthX", 1000);
+ // font.addValueToPrivateDict("nominalWidthX", 0);
font.setFontDict(fontDictionaries);
font.setPrivDict(privateDictionaries);
@@ -1244,15 +1244,6 @@ public class CFFParser
private int nSups;
private Supplement[] supplement;
- /*List<Supplement> getSupplements()
- {
- if (supplement == null)
- {
- return Collections.emptyList();
- }
- return Arrays.asList(supplement);
- }*/
-
/**
* Inner class representing a supplement for an encoding.
*/
Modified:
pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java
URL:
http://svn.apache.org/viewvc/pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java?rev=1667288&r1=1667287&r2=1667288&view=diff
==============================================================================
--- pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java
(original)
+++ pdfbox/trunk/fontbox/src/main/java/org/apache/fontbox/cmap/CMapParser.java
Tue Mar 17 12:29:19 2015
@@ -373,11 +373,7 @@ public class CMapParser
tokenBytes = (byte[]) nextToken;
}
boolean done = false;
- // don't add 1:1 mappings to reduce the memory footprint
- /*if (Arrays.equals(startCode, tokenBytes))
- {
- done = true;
- }*/
+
String value = null;
int arrayIndex = 0;
@@ -574,7 +570,7 @@ public class CMapParser
}
case -1:
{
- // EOF return null;
+ // EOF returning null
break;
}
case '0':
