Followup for #258: Fix forbiddenapis and only print error message if there are actually arguments given.

uschindler · uschindler · commit 6f6ada3d1f36 · 2025-03-31T16:19:30.000+02:00
diff --git a/src/main/java/de/thetaphi/forbiddenapis/cli/CliMain.java b/src/main/java/de/thetaphi/forbiddenapis/cli/CliMain.java
@@ -154,7 +154,9 @@ public CliMain(String... args) throws ExitException {
         throw new ExitException(EXIT_SUCCESS);
       }
     } catch (org.apache.commons.cli.ParseException pe) {
-      System.out.println(pe.getMessage());
+      if (args.length > 0) {
+        StdIoLogger.INSTANCE.error(pe.getMessage());
+      }
       printHelp(options);
       throw new ExitException(EXIT_ERR_CMDLINE);
     }

Original file line number	Diff line number	Diff line change
`@@ -154,7 +154,9 @@ public CliMain(String... args) throws ExitException {`
`154`	`154`	`throw new ExitException(EXIT_SUCCESS);`
`155`	`155`	`}`
`156`	`156`	`} catch (org.apache.commons.cli.ParseException pe) {`
`157`		`- System.out.println(pe.getMessage());`
	`157`	`+ if (args.length > 0) {`
	`158`	`+ StdIoLogger.INSTANCE.error(pe.getMessage());`
	`159`	`+ }`
`158`	`160`	`printHelp(options);`
`159`	`161`	`throw new ExitException(EXIT_ERR_CMDLINE);`
`160`	`162`	`}`