PDFBOX-4892: remove redundant null check for loops, as suggested by valerybokov

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

Author: THausherr

preflight/src/main/java/org/apache/pdfbox/preflight/graphic/ICCProfileWrapper.java

@@ -123,7 +123,11 @@ private static ICCProfileWrapper searchFirstICCProfile(PreflightContext context)
         PDDocumentCatalog catalog = document.getDocumentCatalog();
         COSArray outputIntents = catalog.getCOSObject().getCOSArray(COSName.OUTPUT_INTENTS);
 
-        for (int i = 0; outputIntents != null && i < outputIntents.size(); ++i)
+        if (outputIntents == null)
+        {
+            return null;
+        }
+        for (int i = 0; i < outputIntents.size(); ++i)
         {
             COSDictionary outputIntentDict = (COSDictionary) outputIntents.getObject(i);
             COSBase destOutputProfile = outputIntentDict

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

@@ -192,7 +192,11 @@ private void validateOutputIntent(PreflightContext ctx) throws ValidationExcepti
     {
         COSArray outputIntents = catalog.getCOSObject().getCOSArray(COSName.OUTPUT_INTENTS);
         Map<COSObjectKey, Boolean> tmpDestOutputProfile = new HashMap<>();
-        for (int i = 0; outputIntents != null && i < outputIntents.size(); ++i)
+        if (outputIntents == null)
+        {
+            return;
+        }
+        for (int i = 0; i < outputIntents.size(); ++i)
         {
             COSDictionary outputIntentDict = (COSDictionary) outputIntents.getObject(i);