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 2020/09/01 18:45:59 UTC

svn commit: r1881372 - /pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java

Author: tilman
Date: Tue Sep  1 18:45:59 2020
New Revision: 1881372

URL: http://svn.apache.org/viewvc?rev=1881372&view=rev
Log:
PDFBOX-4892: SonarQube fix: don't return same value in several branches

Modified:
    pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java

Modified: pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java?rev=1881372&r1=1881371&r2=1881372&view=diff
==============================================================================
--- pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java (original)
+++ pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java Tue Sep  1 18:45:59 2020
@@ -295,17 +295,16 @@ public class PDFDebugger extends JFrame
                     List<File> files = (List<File>) transferable.getTransferData(
                             DataFlavor.javaFileListFlavor);
                     readPDFFile(files.get(0), "");
-                    return true;
                 }
                 catch (IOException e)
                 {
                     new ErrorDialog(e).setVisible(true);
-                    return true;
                 }
                 catch (UnsupportedFlavorException e)
                 {
                     throw new RuntimeException(e);
                 }
+                return true;
             }
         });