Author: tilman
Date: Sat Mar 28 13:24:52 2026
New Revision: 1932630
Log:
PDFBOX-5660: replace syserror with dialogbox
Modified:
pdfbox/branches/3.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java
Modified:
pdfbox/branches/3.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java
==============================================================================
---
pdfbox/branches/3.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java
Sat Mar 28 13:24:49 2026 (r1932629)
+++
pdfbox/branches/3.0/debugger/src/main/java/org/apache/pdfbox/debugger/PDFDebugger.java
Sat Mar 28 13:24:52 2026 (r1932630)
@@ -39,7 +39,6 @@ import java.io.File;
import java.io.FileInputStream;
import java.io.IOException;
import java.io.InputStream;
-import java.io.PrintStream;
import java.lang.reflect.Method;
import java.net.URI;
import java.net.URISyntaxException;
@@ -207,10 +206,6 @@ public class PDFDebugger extends JFrame
private String currentTreeViewMode = TreeViewMenu.VIEW_PAGES;
// cli options
- // Expected for CLI app to write to System.out/System.err
- @SuppressWarnings("squid:S106")
- private final PrintStream SYSERR;
-
@Option(names = { "-h", "--help" }, usageHelp = true, description =
"display this help message")
boolean usageHelpRequested;
@@ -233,7 +228,6 @@ public class PDFDebugger extends JFrame
*/
public PDFDebugger()
{
- SYSERR = System.err;
if (viewstructure)
{
currentTreeViewMode = TreeViewMenu.VIEW_STRUCTURE;
@@ -248,14 +242,13 @@ public class PDFDebugger extends JFrame
*/
public PDFDebugger(String initialViewMode)
{
- SYSERR = System.err;
if (TreeViewMenu.isValidViewMode(initialViewMode))
{
currentTreeViewMode = initialViewMode;
}
else
{
- SYSERR.println("Onknown view mode " + initialViewMode);
+ new ErrorDialog(new IllegalArgumentException("Unknown view mode: "
+ "initialViewMode")).setVisible(true);
}
}
@@ -321,7 +314,7 @@ public class PDFDebugger extends JFrame
}
catch (Exception ex)
{
- SYSERR.println( "Error viewing document: " + ex.getMessage());
+ new ErrorDialog(ex).setVisible(true);
return 4;
}
return 0;