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

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-scxml.git


The following commit(s) were added to refs/heads/master by this push:
     new 3812e11  Use isEmpty().
3812e11 is described below

commit 3812e1111ef035e7eaa487430f7c370738c43c31
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Sat Jan 16 21:41:40 2021 -0500

    Use isEmpty().
---
 src/main/java/org/apache/commons/scxml2/NotificationRegistry.java | 2 +-
 src/main/java/org/apache/commons/scxml2/io/ModelUpdater.java      | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/main/java/org/apache/commons/scxml2/NotificationRegistry.java 
b/src/main/java/org/apache/commons/scxml2/NotificationRegistry.java
index 01583ac..10a1697 100644
--- a/src/main/java/org/apache/commons/scxml2/NotificationRegistry.java
+++ b/src/main/java/org/apache/commons/scxml2/NotificationRegistry.java
@@ -73,7 +73,7 @@ public final class NotificationRegistry {
             final Set<SCXMLListener> entries = 
regs.get(source.getObservableId());
             if (entries != null) {
                 entries.remove(lst);
-                if (entries.size() == 0) {
+                if (entries.isEmpty()) {
                     regs.remove(source.getObservableId());
                 }
             }
diff --git a/src/main/java/org/apache/commons/scxml2/io/ModelUpdater.java 
b/src/main/java/org/apache/commons/scxml2/io/ModelUpdater.java
index 9c79903..b5b5815 100644
--- a/src/main/java/org/apache/commons/scxml2/io/ModelUpdater.java
+++ b/src/main/java/org/apache/commons/scxml2/io/ModelUpdater.java
@@ -145,7 +145,7 @@ final class ModelUpdater {
             initialTransition.setNext(scxml.getInitial());
             updateTransition(initialTransition, scxml.getTargets());
 
-            if (initialTransition.getTargets().size() == 0) {
+            if (initialTransition.getTargets().isEmpty()) {
                 logAndThrowModelError(ERR_SCXML_NO_INIT, new Object[] {
                         initial });
             }
@@ -240,7 +240,7 @@ final class ModelUpdater {
             final Set<TransitionTarget> initialStates = 
initialTransition.getTargets();
             // we have to allow for an indirect descendant initial (targets)
             //check that initialState is a descendant of s
-            if (initialStates.size() == 0) {
+            if (initialStates.isEmpty()) {
                 logAndThrowModelError(ERR_STATE_BAD_INIT,
                         new Object[] {getName(state)});
             } else {
@@ -336,7 +336,7 @@ final class ModelUpdater {
         else {
             updateTransition(transition, targets);
             final Set<TransitionTarget> historyStates = 
transition.getTargets();
-            if (historyStates.size() == 0) {
+            if (historyStates.isEmpty()) {
                 logAndThrowModelError(ERR_STATE_NO_HIST,
                         new Object[] {getName(parent)});
             }

Reply via email to