Allon Mureinik has posted comments on this change.

Change subject: core: Remove constant SIZE_IS_NOT_AVAILABLE
......................................................................


Patch Set 1: Code-Review+1

(1 comment)

Code looks good, giving +1 so others can also review.
Please see minor comment regarding the commit message.

https://gerrit.ovirt.org/#/c/39825/1//COMMIT_MSG
Commit Message:

Line 8: 
Line 9: Using null has the same effect, this constant only complicates things.
Line 10: 
Line 11: Change-Id: Ida18353f1d5c976ee70e65045b80410dded2bb1f
Line 12: Related-To: https://bugzilla.redhat.com/1211057
The gerrit hook is throwing a hissy-fit.
Although not technically true, IMHO, I think this should be Bug-Url in order to 
appease the evil process gods.


-- 
To view, visit https://gerrit.ovirt.org/39825
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ida18353f1d5c976ee70e65045b80410dded2bb1f
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Vered Volansky <vvola...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: automat...@ovirt.org
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to