Liron Ar has uploaded a new change for review. Change subject: core: fix findbugs boolean comprasion warning ......................................................................
core: fix findbugs boolean comprasion warning Change-Id: I68e6b0453d8e599e3516de8832096c4b37300fda Signed-off-by: Liron Aravot <[email protected]> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/71/20571/1 diff --git a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java index 0908f88..b09c12c 100644 --- a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java +++ b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/UpdateVmDiskCommand.java @@ -556,7 +556,7 @@ for (Pair<VM, VmDevice> pair : attachedVmsInfo) { VM vm = pair.getFirst(); vmsDiskOrSnapshotAttachedTo.add(vm); - if (pair.getSecond().getIsPlugged() == Boolean.TRUE) { + if (Boolean.TRUE.equals(pair.getSecond().getIsPlugged())) { if (pair.getSecond().getSnapshotId() != null) { vmsDiskSnapshotPluggedTo.add(vm); } else { -- To view, visit http://gerrit.ovirt.org/20571 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I68e6b0453d8e599e3516de8832096c4b37300fda Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Liron Ar <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
