From 44bbe7e2af257469fab2235b5116c81064746f8b Mon Sep 17 00:00:00 2001 From: Robert Varga Date: Thu, 15 Dec 2016 18:22:00 +0100 Subject: [PATCH] Cleanup equals() template Missing a space in 'if(' and has explicit comparison to false, which is not needed. Clean that up. Change-Id: I55f748935070378eab58dd11cf2fb91e8cc9628b Signed-off-by: Robert Varga --- .../config/yangjmxgenerator/plugin/ftl/TemplateFactory.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/opendaylight/config/yang-jmx-generator-plugin/src/main/java/org/opendaylight/controller/config/yangjmxgenerator/plugin/ftl/TemplateFactory.java b/opendaylight/config/yang-jmx-generator-plugin/src/main/java/org/opendaylight/controller/config/yangjmxgenerator/plugin/ftl/TemplateFactory.java index 6e1f54ee53..294520b363 100644 --- a/opendaylight/config/yang-jmx-generator-plugin/src/main/java/org/opendaylight/controller/config/yangjmxgenerator/plugin/ftl/TemplateFactory.java +++ b/opendaylight/config/yang-jmx-generator-plugin/src/main/java/org/opendaylight/controller/config/yangjmxgenerator/plugin/ftl/TemplateFactory.java @@ -439,7 +439,7 @@ public class TemplateFactory { " final %s that = (%s) o;\n", name, name)); for (AttributeIfc s : attrs.values()) { equalsBodyBuilder.append(String.format( - " if(java.util.Objects.equals(%1$s, that.%1$s) == false) {\n" + + " if (!java.util.Objects.equals(%1$s, that.%1$s)) {\n" + " return false;\n" + " }\n\n", s.getLowerCaseCammelCase())); } -- 2.36.6