Simplify empty checking 73/88673/1
authorRobert Varga <robert.varga@pantheon.tech>
Wed, 25 Mar 2020 14:34:22 +0000 (15:34 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Wed, 25 Mar 2020 14:34:58 +0000 (15:34 +0100)
Use a simple negation instead of an equality check -- not that it
makes much difference in generated code.

Change-Id: If7c73a3884544c5acb5590b8154df712d4af2c55
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/mdsal/binding/java/api/generator/ClassTemplate.xtend

index 0f5620597994d7326b60f0ab86745aa4ba79f596..ca39faad6e4b4bdeb42ed655bb7b252d2fa332f8 100644 (file)
@@ -217,7 +217,7 @@ class ClassTemplate extends BaseTemplate {
 
     def private allValuesConstructor() '''
     public «type.name»(«allProperties.asArgumentsDeclaration») {
-        «IF false == parentProperties.empty»
+        «IF !parentProperties.empty»
             super(«parentProperties.asArguments»);
         «ENDIF»
         «FOR p : allProperties»
@@ -239,7 +239,7 @@ class ClassTemplate extends BaseTemplate {
     @«ConstructorParameters.importedName»("«TypeConstants.VALUE_PROP»")
     @«ConstructorProperties.importedName»("«TypeConstants.VALUE_PROP»")
     public «type.name»(«allProperties.asArgumentsDeclaration») {
-        «IF false == parentProperties.empty»
+        «IF !parentProperties.empty»
             super(«parentProperties.asArguments»);
         «ENDIF»
         «FOR p : allProperties»
@@ -274,7 +274,7 @@ class ClassTemplate extends BaseTemplate {
 
     def protected genConstructor(GeneratedProperty property, Iterable<GeneratedProperty> other) '''
     public «type.name»(«property.returnType.importedName + " " + property.name») {
-        «IF false == parentProperties.empty»
+        «IF !parentProperties.empty»
             super(«parentProperties.asArguments»);
         «ENDIF»
 
@@ -327,7 +327,7 @@ class ClassTemplate extends BaseTemplate {
      * @param source Source object
      */
     public «type.name»(«type.name» source) {
-        «IF false == parentProperties.empty»
+        «IF !parentProperties.empty»
             super(source);
         «ENDIF»
         «FOR p : properties»