Change in ovirt-engine[master]: webadmin: Remove useless condition in "PolicyUnitPanel.initW...

juan.hernandez at redhat.com juan.hernandez at redhat.com
Thu Sep 24 11:34:06 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: webadmin: Remove useless condition in "PolicyUnitPanel.initWidget"
......................................................................


webadmin: Remove useless condition in "PolicyUnitPanel.initWidget"

This is necessary to avoid the following findbugs error:

  Condition has no effect

  This condition always produces the same result as the value of the
  involved variable was narrowed before. Probably something else was
  meant or condition can be removed.

  Bug kind and pattern: UC - UC_USELESS_CONDITION

Change-Id: I99b0cfdc5f098fdabea9680fe823567592b83a53
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M frontend/webadmin/modules/webadmin/src/main/java/org/ovirt/engine/ui/webadmin/section/main/view/popup/scheduling/panels/PolicyUnitPanel.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Juan Hernandez: Verified; Passed CI tests
  Alexander Wels: Looks good to me, approved



-- 
To view, visit https://gerrit.ovirt.org/46557
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I99b0cfdc5f098fdabea9680fe823567592b83a53
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list