[royale-compiler] 02/05: formatter: fix problems not being printed for configuration options

[joshtynjala]

This is an automated email from the ASF dual-hosted git repository.

joshtynjala pushed a commit to branch develop
in repository https://gitbox.apache.org/repos/asf/royale-compiler.git

commit 7c7ac387c2d204976f7e858620d90a750115a298
Author: Josh Tynjala <joshtynj...@apache.org>
AuthorDate: Wed May 18 09:57:08 2022 -0700

    formatter: fix problems not being printed for configuration options
---
 .../src/main/java/org/apache/royale/formatter/FORMATTER.java     | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/formatter/src/main/java/org/apache/royale/formatter/FORMATTER.java 
b/formatter/src/main/java/org/apache/royale/formatter/FORMATTER.java
index 38f429fe7..d6fae8340 100644
--- a/formatter/src/main/java/org/apache/royale/formatter/FORMATTER.java
+++ b/formatter/src/main/java/org/apache/royale/formatter/FORMATTER.java
@@ -142,7 +142,7 @@ public class FORMATTER {
                problems = new ProblemQuery();
 
                try {
-                       boolean continueFormatting = configure(args);
+                       boolean continueFormatting = configure(args, problems);
                        if (continueFormatting) {
                                if (inputFiles.size() == 0) {
                                        StringBuilder builder = new 
StringBuilder();
@@ -294,10 +294,8 @@ public class FORMATTER {
                System.out.println(usages);
        }
 
-       private boolean configure(String[] args) {
+       private boolean configure(String[] args, ProblemQuery problems) {
                try {
-                       problems = new ProblemQuery();
-
                        Configurator configurator = new Configurator();
                        configurator.setConfiguration(args, "files");
                        configuration = configurator.getConfiguration();
@@ -317,8 +315,9 @@ public class FORMATTER {
                                return false;
                        }
 
-                       if (problems.hasErrors())
+                       if (problems.hasErrors()) {
                                return false;
+                       }
 
                        collapseEmptyBlocks = 
configuration.getCollapseEmptyBlocks();
                        ignoreProblems = 
configuration.getIgnoreParsingProblems();