Fix checkstyle if-statements must use braces in yang-parser-impl

[yangtools.git] / yang / yang-parser-impl / src / main / java / org / opendaylight / yangtools / yang / parser / builder / impl / RefineUtils.java

diff --git a/yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/RefineUtils.java b/yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/RefineUtils.java
index c5071762f15dd81b9acc048c8825f83e224a7295..e3f02b8dc265471afb28c70c27ce356366d1b119 100644 (file)
--- a/yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/RefineUtils.java
+++ b/yang/yang-parser-impl/src/main/java/org/opendaylight/yangtools/yang/parser/builder/impl/RefineUtils.java
@@ -8,7 +8,6 @@
 package org.opendaylight.yangtools.yang.parser.builder.impl;
 
 import java.util.List;
-
 import org.opendaylight.yangtools.yang.model.api.MustDefinition;
 import org.opendaylight.yangtools.yang.parser.builder.api.Builder;
 import org.opendaylight.yangtools.yang.parser.builder.api.DataSchemaNodeBuilder;
@@ -30,13 +29,13 @@ public final class RefineUtils {
     private RefineUtils() {
     }
 
-    public static void refineLeaf(final LeafSchemaNodeBuilder leaf, final RefineBuilder refine) {
+    private static void refineLeaf(final LeafSchemaNodeBuilder leaf, final RefineBuilder refine) {
         String defaultStr = refine.getDefaultStr();
         Boolean mandatory = refine.isMandatory();
         MustDefinition must = refine.getMust();
         List<UnknownSchemaNodeBuilder> unknownNodes = refine.getUnknownNodes();
 
-        if (defaultStr != null && !("".equals(defaultStr))) {
+        if (defaultStr != null && !defaultStr.isEmpty()) {
             leaf.setDefaultStr(defaultStr);
         }
         if (mandatory != null) {
@@ -53,7 +52,7 @@ public final class RefineUtils {
         }
     }
 
-    public static void refineContainer(final ContainerSchemaNodeBuilder container, final RefineBuilder refine) {
+    private static void refineContainer(final ContainerSchemaNodeBuilder container, final RefineBuilder refine) {
         Boolean presence = refine.isPresence();
         MustDefinition must = refine.getMust();
         List<UnknownSchemaNodeBuilder> unknownNodes = refine.getUnknownNodes();
@@ -72,7 +71,7 @@ public final class RefineUtils {
         }
     }
 
-    public static void refineList(final ListSchemaNodeBuilder list, final RefineBuilder refine) {
+    private static void refineList(final ListSchemaNodeBuilder list, final RefineBuilder refine) {
         MustDefinition must = refine.getMust();
         Integer min = refine.getMinElements();
         Integer max = refine.getMaxElements();
@@ -187,7 +186,6 @@ public final class RefineUtils {
         } else if (node instanceof ContainerSchemaNodeBuilder) {
             checkRefineDefault(node, defaultStr, moduleName, line);
             checkRefineMandatory(node, mandatory, moduleName, line);
-            checkRefineMust(node, must, moduleName, line);
             checkRefineMinMax(name, min, max, moduleName, line);
         } else if (node instanceof LeafSchemaNodeBuilder) {
             checkRefinePresence(node, presence, moduleName, line);
@@ -259,7 +257,6 @@ public final class RefineUtils {
     private static void refineDefault(final Builder node, final RefineBuilder refine) {
         final String moduleName = refine.getModuleName();
         final int line = refine.getLine();
-        Class<? extends Builder> cls = node.getClass();
 
 
         final DocumentedNodeBuilder documentedNode;