Yair Zaslavsky has posted comments on this change.
Change subject: engine: Unify Guid and NGuid
......................................................................
Patch Set 2: (1 inline comment)
....................................................
File
backend/manager/modules/compat/src/main/java/org/ovirt/engine/core/compat/Guid.java
Line 63: return new Guid(candidate);
Line 64: }
Line 65: }
Line 66:
Line 67: public static Guid createGuidFromStringDefaultEmpty(String
candidate) {
I don't like this method name.
Perhaps -
createGuidFromString(String candidate, boolean useEmptyIfNull) - and have
createGuidFromString(String candidate) call
createGuidFromString(candidate,false) and use
createGuidFromString(candidate,true) in all places you intended to use the
method that ends with DefaultEmpty?
Line 68: if (candidate == null) {
Line 69: return Guid.Empty;
Line 70: } else {
Line 71: return new Guid(candidate);
--
To view, visit http://gerrit.ovirt.org/15691
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I4af6d353ef27394ff902f374de19682628c52739
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Alissa Bonas <abo...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Daniel Erez <de...@redhat.com>
Gerrit-Reviewer: Liron Ar <lara...@redhat.com>
Gerrit-Reviewer: Maor Lipchuk <mlipc...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: Omer Frenkel <ofren...@redhat.com>
Gerrit-Reviewer: Sergey Gotliv <sgot...@redhat.com>
Gerrit-Reviewer: Tal Nisan <tni...@redhat.com>
Gerrit-Reviewer: Vered Volansky <vvola...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
