Change in ovirt-engine[master]: webadmin: Remove useless condition in "VmBrickModel.validate...
Juan Hernandez has submitted this change and it was merged.
Change subject: webadmin: Remove useless condition in
"VmBrickModel.validateBrickCount"
......................................................................
webadmin: Remove useless condition in "VmBrickModel.validateBrickCount"
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: I99bb4ac3912e250d1406d9426ada2c79d956a8b2
Signed-off-by: Juan Hernandez <juan.hernandez(a)redhat.com>
---
M
frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/gluster/VolumeBrickModel.java
1 file changed, 1 insertion(+), 3 deletions(-)
Approvals:
Juan Hernandez: Verified; Passed CI tests
Alexander Wels: Looks good to me, approved
--
To view, visit https://gerrit.ovirt.org/46554
To unsubscribe, visit https://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I99bb4ac3912e250d1406d9426ada2c79d956a8b2
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Alexander Wels <awels(a)redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez(a)redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org