Minor fixes to new code (JENA-1434)

- Logic error in hadErrors() and hadWarnings() - Minor formatting
OpenSensingCity · Nov 28, 2017 · f733d88 · f733d88
1 parent bb0542e
commit f733d88
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/jena-arq/src/main/java/org/apache/jena/riot/system/ErrorHandlerFactory.java b/jena-arq/src/main/java/org/apache/jena/riot/system/ErrorHandlerFactory.java
@@ -270,11 +270,11 @@ public long getWarningCount() {
         }
 
         public boolean hadErrors() {
-            return this.errorCount > 1;
+            return this.errorCount > 0;
         }
 
         public boolean hadWarnings() {
-            return this.warningCount > 1;
+            return this.warningCount > 0;
         }
 
         public boolean hadIssues() {

diff --git a/jena-cmds/src/main/java/riotcmd/CmdLangParse.java b/jena-cmds/src/main/java/riotcmd/CmdLangParse.java
@@ -244,7 +244,9 @@ protected ParseRecord parseRIOT(RDFParserBuilder builder, /*Info for the Process
             checking = false;
         builder.checking(checking);
 
-        ErrorHandlerTracking errHandler = ErrorHandlerFactory.errorHandlerTracking(ErrorHandlerFactory.stdLogger, modLangParse.stopOnBadTerm(), modLangParse.stopOnWarnings());
+        ErrorHandlerTracking errHandler = ErrorHandlerFactory.errorHandlerTracking(ErrorHandlerFactory.stdLogger, 
+                                                                                   modLangParse.stopOnBadTerm(), 
+                                                                                   modLangParse.stopOnWarnings());
 
         if ( modLangParse.skipOnBadTerm() ) {
             // skipOnBadterm - this needs collaboration from the parser.