]> source.dussan.org Git - vaadin-framework.git/commitdiff
Use Collection .isEmpty() over .size() == 0
authorPer-Åke Minborg <minborg@speedment.com>
Fri, 28 Oct 2016 16:01:19 +0000 (09:01 -0700)
committerVaadin Code Review <review@vaadin.com>
Fri, 4 Nov 2016 13:36:29 +0000 (13:36 +0000)
Change-Id: I80b73b653e97904605dc62484a7448f3bfbf721f

server/src/main/java/com/vaadin/server/CompositeErrorMessage.java
server/src/main/java/com/vaadin/server/JsonPaintTarget.java
server/src/main/java/com/vaadin/ui/declarative/Design.java

index 6e37a94c5e4926b6780faa73df39b38f36bf3eaa..f96bcbac279ee14117e01075652a87f36f4f11e9 100644 (file)
@@ -43,7 +43,7 @@ public class CompositeErrorMessage extends AbstractErrorMessage {
             addErrorMessage(errorMessage);
         }
 
-        if (getCauses().size() == 0) {
+        if (getCauses().isEmpty()) {
             throw new IllegalArgumentException(
                     "Composite error message must have at least one error");
         }
@@ -67,7 +67,7 @@ public class CompositeErrorMessage extends AbstractErrorMessage {
             addErrorMessage(i.next());
         }
 
-        if (getCauses().size() == 0) {
+        if (getCauses().isEmpty()) {
             throw new IllegalArgumentException(
                     "Composite error message must have at least one error");
         }
index 9792673062597e00b3ded8b33d7ce218002781cb..d5276aefaf9c5d4d8ea83fb4ad2e8f01d114c9e1 100644 (file)
@@ -838,7 +838,7 @@ public class JsonPaintTarget implements PaintTarget {
         }
 
         private String variablesAsJsonObject() {
-            if (variables.size() == 0) {
+            if (variables.isEmpty()) {
                 return "";
             }
             final StringBuilder buf = new StringBuilder();
index 0c5595028ef3970e28e09d180f5db0c976439f12..b5b6762f209c853a8d497a261d8306353ec12d4d 100644 (file)
@@ -454,7 +454,7 @@ public class Design implements Serializable {
                     "The first level of a component hierarchy should contain at most one root component, but found "
                             + children.size() + ".");
         }
-        Element element = children.size() == 0 ? null : children.first();
+        Element element = children.isEmpty() ? null : children.first();
         if (componentRoot != null) {
             if (element == null) {
                 throw new DesignException(