From 5626a340dd271ec585f973dfa99a2a7fb45abac3 Mon Sep 17 00:00:00 2001
From: Laszlo Hornyak <laszlo.hornyak@gmail.com>
Date: Wed, 25 Feb 2015 20:55:56 +0100
Subject: [PATCH] Logical expression simplification, == true is redundant

Signed-off-by: Laszlo Hornyak <laszlo.hornyak@gmail.com>
---
 .../com/cloud/agent/api/routing/SetFirewallRulesCommand.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/core/src/com/cloud/agent/api/routing/SetFirewallRulesCommand.java b/core/src/com/cloud/agent/api/routing/SetFirewallRulesCommand.java
index be85887f36c..e0305453068 100644
--- a/core/src/com/cloud/agent/api/routing/SetFirewallRulesCommand.java
+++ b/core/src/com/cloud/agent/api/routing/SetFirewallRulesCommand.java
@@ -53,7 +53,7 @@ public class SetFirewallRulesCommand extends NetworkElementCommand {
              *  each entry format      <ip>:protocol:srcport:destport:scidr:
              *  reverted entry format  <ip>:reverted:0:0:0:
              */
-            if (fwTO.revoked() == true) {
+            if (fwTO.revoked()) {
                 StringBuilder sb = new StringBuilder();
                 /* This entry is added just to make sure atleast there will one entry in the list to get the ipaddress */
                 sb.append(fwTO.getSrcIp()).append(":reverted:0:0:0:");