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 2014/12/23 22:21:51 UTC
svn commit: r1647677 -
/pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/FontMetaDataValidation.java
Author: tilman
Date: Tue Dec 23 21:21:51 2014
New Revision: 1647677
URL: http://svn.apache.org/r1647677
Log:
PDFBOX-2576: simplify code
Modified:
pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/FontMetaDataValidation.java
Modified: pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/FontMetaDataValidation.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/FontMetaDataValidation.java?rev=1647677&r1=1647676&r2=1647677&view=diff
==============================================================================
--- pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/FontMetaDataValidation.java (original)
+++ pdfbox/trunk/preflight/src/main/java/org/apache/pdfbox/preflight/font/util/FontMetaDataValidation.java Tue Dec 23 21:21:51 2014
@@ -100,10 +100,10 @@ public class FontMetaDataValidation
{
empty = false;
AbstractField tmp = it.next();
- if (tmp != null && tmp instanceof TextType)
+ if (tmp instanceof TextType)
{
- if (((TextType) tmp).getStringValue().equals(fontName)
- || (noSubSetName != null && ((TextType) tmp).getStringValue().equals(noSubSetName)))
+ String val = ((TextType) tmp).getStringValue();
+ if (val.equals(fontName) || val.equals(noSubSetName))
{
// value found, return
return true;