Doron Fediuck has submitted this change and it was merged. Change subject: core: clean up VDSGroup.equals() ...................................................................... core: clean up VDSGroup.equals() VDSGroup.equals() was missing braces for many if statements. During cleanup, found it was also missing a return statement and thus didn't properly compare description and high_utilization. Change-Id: I1036db064d343ddc70ab49eeefa0e94e6d2b0f2e Signed-off-by: Greg Padgett <gpadgett@redhat.com> --- M backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/businessentities/VDSGroup.java 1 file changed, 57 insertions(+), 28 deletions(-) Approvals: Doron Fediuck: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/12061 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1036db064d343ddc70ab49eeefa0e94e6d2b0f2e Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Greg Padgett <gpadgett@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Doron Fediuck <dfediuck@redhat.com> Gerrit-Reviewer: Greg Padgett <gpadgett@redhat.com>