X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=binding%2Fmdsal-binding-java-api-generator%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fyangtools%2Fsal%2Fjava%2Fapi%2Fgenerator%2FBuilderTemplate.xtend;h=300d725f97847938ce89cc39b76ebaf324930264;hb=85de37f352faa1ef6319db3141a68a99b6c9f365;hp=126c3ed46194c9bd59fc4a785279b1c16f24fa08;hpb=d92aa291ec070cfaf7b680d99f015f4402dcfe60;p=mdsal.git

diff --git a/binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/yangtools/sal/java/api/generator/BuilderTemplate.xtend b/binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/yangtools/sal/java/api/generator/BuilderTemplate.xtend
index 126c3ed461..300d725f97 100644
--- a/binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/yangtools/sal/java/api/generator/BuilderTemplate.xtend
+++ b/binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/yangtools/sal/java/api/generator/BuilderTemplate.xtend
@@ -71,10 +71,6 @@ class BuilderTemplate extends BaseTemplate {
     val Set<GeneratedProperty> properties
 
     private static val METHOD_COMPARATOR = new AlphabeticallyTypeMemberComparator<MethodSignature>();
-    private static val APPEND_COMMA = "builder.append(\", \");";
-    private static val APPEND_COMMA_WITH_WS = "\n    "+ APPEND_COMMA + "\n}";
-    private static val APPEND_ENDIF = "    }\n";
-    private static val APPEND_END = "return builder.append(']').toString();";
 
     /**
      * Constructs new instance of this class.
@@ -715,29 +711,33 @@ class BuilderTemplate extends BaseTemplate {
             @Override
             public «String.importedName» toString() {
                 «StringBuilder.importedName» builder = new «StringBuilder.importedName» ("«type.name» [");
-                «FOR property : properties SEPARATOR APPEND_COMMA_WITH_WS AFTER APPEND_ENDIF»
+                boolean first = true;
+
+                «FOR property : properties»
                     if («property.fieldName» != null) {
+                        if (first) {
+                            first = false;
+                        } else {
+                            builder.append(", ");
+                        }
                         builder.append("«property.fieldName»=");
                         «IF property.returnType.name.contains("[")»
                             builder.append(«Arrays.importedName».toString(«property.fieldName»));
                         «ELSE»
                             builder.append(«property.fieldName»);
                         «ENDIF»
+                     }
                 «ENDFOR»
                 «IF augmentField != null»
-                    «IF !properties.empty»
-                «APPEND_COMMA»
-                    «ENDIF»
+                    if (first) {
+                        first = false;
+                    } else {
+                        builder.append(", ");
+                    }
                     builder.append("«augmentField.name»=");
-                    builder.append(«augmentField.name».values());«"\n"»
-                    «APPEND_END»
-                «ELSE»
-                    «IF properties.empty»
-                    «APPEND_END»
-                    «ELSE»
-            «APPEND_END»
-                    «ENDIF»
+                    builder.append(«augmentField.name».values());
                 «ENDIF»
+                return builder.append(']').toString();
             }
         «ENDIF»
     '''