Change order of constructors

Minor changes in test function

DEVSIX-5705

Author: Egor Martsynkovsky

kernel/src/main/java/com/itextpdf/kernel/pdf/PdfReader.java

@@ -122,12 +122,6 @@ public class PdfReader implements Closeable, Serializable {
     protected boolean hybridXref = false;
     protected boolean fixedXref = false;
     protected boolean xrefStm = false;
-
-    PdfReader(IRandomAccessSource byteSource, ReaderProperties properties, boolean closeStream) throws IOException {
-        this.properties = properties;
-        this.tokens = getOffsetTokeniser(byteSource, closeStream);
-    }
-    
     /**
      * Constructs a new PdfReader.
      *
@@ -202,6 +196,11 @@ public PdfReader(String filename) throws IOException {
 
     }
 
+    PdfReader(IRandomAccessSource byteSource, ReaderProperties properties, boolean closeStream) throws IOException {
+        this.properties = properties;
+        this.tokens = getOffsetTokeniser(byteSource, closeStream);
+    }
+
     /**
      * Close {@link PdfTokenizer}.
      *

kernel/src/test/java/com/itextpdf/kernel/pdf/PdfReaderTest.java

@@ -2045,7 +2045,8 @@ public void notCloseUserStreamTest() throws IOException {
             Assert.assertEquals(-1, pdfStream.read());
         }
     }
-    private File copyFileForTest(String fileName, String copiedFileName) throws IOException {
+    
+    private static File copyFileForTest(String fileName, String copiedFileName) throws IOException {
         File copiedFile = new File(copiedFileName);
         Files.copy(Paths.get(fileName), Paths.get(copiedFileName));
         return copiedFile;