Michael Pasternak has posted comments on this change. Change subject: restapi: #868626 - All-Content Header ......................................................................
Patch Set 2: I would prefer that you didn't submit this i'd suggest instead of in every place populate() get called, using if(isPopulate), inside of populate method adding if(isPopulate), this will make code much more clear and won't require using 'if(isPopulate)' in every place that calling populate() -- To view, visit http://gerrit.ovirt.org/9018 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I9ecfe2f374599eafecf9c683038b3eacfd4bca50 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Ori Liel <ol...@redhat.com> Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com> Gerrit-Reviewer: Ori Liel <ol...@redhat.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
