Juan Hernandez has posted comments on this change.

Change subject: [WIP] Introduce new directory interface
......................................................................


Patch Set 4: (1 inline comment)

....................................................
File 
backend/manager/modules/auth/src/main/java/org/ovirt/engine/core/directory/DirectoryUser.java
Line 114:             return true;
Line 115:         }
Line 116:         if (obj == null) {
Line 117:             return false;
Line 118:         }
I will change this.
Line 119:         if (getClass() != obj.getClass()) {
Line 120:             return false;
Line 121:         }
Line 122:         if (!super.equals(obj)) {


-- 
To view, visit http://gerrit.ovirt.org/15596
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: If84a0c9d6553d81cdbbe224972696f169cca90d4
Gerrit-PatchSet: 4
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Juan Hernandez <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Ravi Nori <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to