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 2021/03/02 17:09:14 UTC

svn commit: r1887095 - /pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSArray.java

Author: tilman
Date: Tue Mar  2 17:09:14 2021
New Revision: 1887095

URL: http://svn.apache.org/viewvc?rev=1887095&view=rev
Log:
PDFBOX-4892: adjust type + remove unneeded code, as suggested by valerybokov

Modified:
    pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSArray.java

Modified: pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSArray.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSArray.java?rev=1887095&r1=1887094&r2=1887095&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSArray.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/main/java/org/apache/pdfbox/cos/COSArray.java Tue Mar  2 17:09:14 2021
@@ -185,7 +185,7 @@ public class COSArray extends COSBase im
      */
     public COSBase getObject( int index )
     {
-        Object obj = objects.get( index );
+        COSBase obj = objects.get( index );
         if( obj instanceof COSObject )
         {
             obj = ((COSObject)obj).getObject();
@@ -194,7 +194,7 @@ public class COSArray extends COSBase im
         {
             obj = null;
         }
-        return (COSBase)obj;
+        return obj;
     }
 
     /**
@@ -456,7 +456,7 @@ public class COSArray extends COSBase im
     public int indexOfObject(COSBase object)
     {
         int retval = -1;
-        for (int i = 0; retval < 0 && i < this.size(); i++)
+        for (int i = 0; i < this.size(); i++)
         {
             COSBase item = this.get(i);
             if (item.equals(object) ||