svn commit: r1932637 - in pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger: . pagepane

[tilman]

Author: tilman
Date: Sat Mar 28 14:21:34 2026
New Revision: 1932637

Log:
PDFBOX-5660: replace runtime throw; remove unintended quotes

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

Modified: 
pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java
==============================================================================
--- 
pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java
      Sat Mar 28 14:18:50 2026        (r1932636)
+++ 
pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java
      Sat Mar 28 14:21:34 2026        (r1932637)
@@ -230,9 +230,9 @@ public class PDFDebugger extends JFrame
                 configuration.load(is);
                 is.close();
             }
-            catch(IOException e)
+            catch (IOException ex)
             {
-                throw new RuntimeException(e);
+                new ErrorDialog(ex).setVisible(true);
             }
         }
     }
@@ -376,9 +376,9 @@ public class PDFDebugger extends JFrame
                 osxQuit.setAccessible(true);
                 OSXAdapter.setQuitHandler(this, osxQuit);
             }
-            catch (NoSuchMethodException e)
+            catch (NoSuchMethodException ex)
             {
-                throw new RuntimeException(e);
+                new ErrorDialog(ex).setVisible(true);
             }
         }
     }
@@ -416,9 +416,9 @@ public class PDFDebugger extends JFrame
                 {
                     readPDFurl(urlString, "");
                 }
-                catch (IOException e)
+                catch (IOException ex)
                 {
-                    throw new RuntimeException(e);
+                    new ErrorDialog(ex).setVisible(true);
                 }
             }
         });
@@ -1238,7 +1238,7 @@ public class PDFDebugger extends JFrame
             }
             catch (IOException e)
             {
-                throw new RuntimeException(e);
+                new ErrorDialog(ex).setVisible(true);
             }
             finally
             {
@@ -1622,9 +1622,9 @@ public class PDFDebugger extends JFrame
                 {
                     readPDFFile(filePath, "");
                 }
-                catch (Exception e)
+                catch (Exception ex)
                 {
-                    throw new RuntimeException(e);
+                    new ErrorDialog(ex).setVisible(true);
                 }
             }
         };

Modified: 
pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/pagepane/PagePane.java
==============================================================================
--- 
pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/pagepane/PagePane.java
        Sat Mar 28 14:18:50 2026        (r1932636)
+++ 
pdfbox/branches/2.0/debugger/src/main/java/org/apache/pdfbox/debugger/pagepane/PagePane.java
        Sat Mar 28 14:21:34 2026        (r1932637)
@@ -486,15 +486,15 @@ public class PagePane implements ActionL
                 label.setIcon(new HighResolutionImageIcon(image, 
label.getWidth(), label.getHeight()));
                 label.setText(null);
             }
-            catch (InterruptedException e)
+            catch (InterruptedException ex)
             {
-                label.setText(e.getMessage());
-                throw new RuntimeException(e);
+                label.setText(ex.getMessage());
+                new ErrorDialog(ex).setVisible(true);
             }
-            catch (ExecutionException e)
+            catch (ExecutionException ex)
             {
-                label.setText(e.getMessage());
-                throw new RuntimeException(e);
+                label.setText(ex.getMessage());
+                new ErrorDialog(ex).setVisible(true);
             }
         }
     }