Alexander Wels has posted comments on this change. Change subject: webadmin: System tree refresh ......................................................................
Patch Set 3: (3 comments) http://gerrit.ovirt.org/#/c/26161/3/frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SystemTreeItemModel.java File frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SystemTreeItemModel.java: Line 131: } Line 132: if (result) { Line 133: if ((otherModel.getEntity() == null && getEntity() == null) Line 134: || (otherModel.getEntity() != null && otherModel.getEntity().equals(getEntity()) Line 135: && otherModel.getTitle().equals(getTitle()))) { > I know that it's unlikely for the title attribute to be null, but to honor Done Line 136: result = true; Line 137: } else { Line 138: result = false; Line 139: } Line 132: if (result) { Line 133: if ((otherModel.getEntity() == null && getEntity() == null) Line 134: || (otherModel.getEntity() != null && otherModel.getEntity().equals(getEntity()) Line 135: && otherModel.getTitle().equals(getTitle()))) { Line 136: result = true; > Hm, this seems a bit strange to me, as result is already true here. Done Line 137: } else { Line 138: result = false; Line 139: } Line 140: } Line 145: public int hashCode() { Line 146: final int prime = 31; Line 147: int result = 1; Line 148: result = prime * result + getTitle().hashCode(); Line 149: result = prime * result + super.hashCode(); > I'd suggest to consider conventional hashCode implementation that is consis Done Line 150: return result; Line 151: } Line 152: -- To view, visit http://gerrit.ovirt.org/26161 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I062163b1f6d2e5fcfd1539b69a3d0d2cee674654 Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Alexander Wels <aw...@redhat.com> Gerrit-Reviewer: Alexander Wels <aw...@redhat.com> Gerrit-Reviewer: Einav Cohen <eco...@redhat.com> Gerrit-Reviewer: Greg Sheremeta <gsher...@redhat.com> Gerrit-Reviewer: Lior Vernia <lver...@redhat.com> Gerrit-Reviewer: Vojtech Szocs <vsz...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-Reviewer: oVirt Jenkins CI Server Gerrit-HasComments: Yes _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
