PDFBOX-4892: remove unneeded code, as suggested by valerybokov

git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1891446 13f79535-47bb-0310-9956-ffa450edef68

Author: THausherr

preflight/src/main/java/org/apache/pdfbox/preflight/process/AcroFormValidationProcess.java

@@ -37,11 +37,9 @@
 import org.apache.pdfbox.preflight.exception.ValidationException;
 import org.apache.pdfbox.preflight.utils.ContextHelper;
 
-
 import static org.apache.pdfbox.preflight.PreflightConfiguration.ANNOTATIONS_PROCESS;
 import static org.apache.pdfbox.preflight.PreflightConstants.ERROR_ACTION_FORBIDDEN_ADDITIONAL_ACTIONS_FIELD;
 import static org.apache.pdfbox.preflight.PreflightConstants.ERROR_ACTION_FORBIDDEN_WIDGET_ACTION_FIELD;
-import static org.apache.pdfbox.preflight.PreflightConstants.ERROR_SYNTAX_BODY;
 import static org.apache.pdfbox.preflight.PreflightConstants.ERROR_SYNTAX_DICT_INVALID;
 import static org.apache.pdfbox.preflight.PreflightConstants.ERROR_SYNTAX_NOCATALOG;
 
@@ -97,34 +95,17 @@ protected void checkNeedAppearences(PreflightContext ctx, PDAcroForm acroForm)
      * @see #validateField(PreflightContext, PDField) 
      * 
      * @param ctx the preflight context.
-     * @param lFields the list of fields, can be null.
+     * @param fields the list of fields, never null.
      * @return the result of the validation.
      * @throws IOException
      */
-    protected boolean exploreFields(PreflightContext ctx, List<PDField> lFields) throws IOException
+    protected boolean exploreFields(PreflightContext ctx, List<PDField> fields) throws IOException
     {
-        if (lFields != null)
+        for (PDField field : fields)
         {
-            // the list can be null if the field doesn't have children
-            for (Object obj : lFields)
+            if (!validateField(ctx, field))
             {
-                if (obj instanceof PDField)
-                {
-                    if (!validateField(ctx, (PDField) obj))
-                    {
-                        return false;
-                    }
-                }
-                else if (obj instanceof PDAnnotationWidget)
-                {
-                    // "A field's children in the hierarchy may also include widget annotations"
-                    ContextHelper.validateElement(ctx, ((PDAnnotationWidget) obj).getCOSObject(), ANNOTATIONS_PROCESS);
-                }
-                else
-                {
-                    addValidationError(ctx, new ValidationError(ERROR_SYNTAX_BODY,
-                            "Field can only have fields or widget annotations as KIDS"));
-                }
+                return false;
             }
         }
         return true;