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/03/17 17:14:36 UTC

svn commit: r1787453 - /pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDField.java

Author: tilman
Date: Fri Mar 17 17:14:36 2017
New Revision: 1787453

URL: http://svn.apache.org/viewvc?rev=1787453&view=rev
Log:
PDFBOX-2852: remove condition that is always true; remove empty statement

Modified:
    pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDField.java

Modified: pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDField.java
URL: http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDField.java?rev=1787453&r1=1787452&r2=1787453&view=diff
==============================================================================
--- pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDField.java (original)
+++ pdfbox/trunk/pdfbox/src/main/java/org/apache/pdfbox/pdmodel/interactive/form/PDField.java Fri Mar 17 17:14:36 2017
@@ -247,7 +247,7 @@ public abstract class PDField implements
             
             if (fieldValue instanceof COSName)
             {
-                currentField.setValue(((COSName) fieldValue).getName());;
+                currentField.setValue(((COSName) fieldValue).getName());
             }
             else if (fieldValue instanceof COSString)
             {
@@ -261,7 +261,7 @@ public abstract class PDField implements
             {
                 ((PDChoice) this).setValue(COSArrayList.convertCOSStringCOSArrayToList((COSArray) fieldValue));
             }
-            else if (fieldValue != null)
+            else
             {
                 throw new IOException("Error:Unknown type for field import" + fieldValue);
             }