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 2016/03/25 17:42:00 UTC
svn commit: r1736615 -
/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java
Author: lehmi
Date: Fri Mar 25 16:42:00 2016
New Revision: 1736615
URL: http://svn.apache.org/viewvc?rev=1736615&view=rev
Log:
PDFBOX-3286: fixed constant value and constant usage as proposed by Christophe Bouchon
Modified:
pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java
Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java?rev=1736615&r1=1736614&r2=1736615&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/font/TrueTypeEmbedder.java Fri Mar 25 16:42:00 2016
@@ -50,7 +50,7 @@ import org.apache.pdfbox.pdmodel.common.
abstract class TrueTypeEmbedder implements Subsetter
{
private static final int ITALIC = 1;
- private static final int OBLIQUE = 256;
+ private static final int OBLIQUE = 512;
private static final String BASE25 = "BCDEFGHIJKLMNOPQRSTUVWXYZ";
private final PDDocument document;
@@ -185,8 +185,7 @@ abstract class TrueTypeEmbedder implemen
ttf.getHorizontalHeader().getNumberOfHMetrics() == 1);
int fsSelection = os2.getFsSelection();
- fd.setItalic((fsSelection & ITALIC) == fsSelection ||
- (fsSelection & OBLIQUE) == fsSelection);
+ fd.setItalic(((fsSelection & (ITALIC | OBLIQUE)) != 0));
switch (os2.getFamilyClass())
{