Allon Mureinik has posted comments on this change. Change subject: core: Cleanup VmPoolCommandBase#isVmFree ......................................................................
Patch Set 2: Verified; Looks good to me, approved Rebased, fixed grammer issues noted by Tal. re: moving the code to some place sensible - let's first make it (somewhat) readable, so when someone does put the time and effort to move it where it deserves (/dev/null, probably), it will be doable. -- To view, visit http://gerrit.ovirt.org/12124 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2e59cd811d7c318674587df877d29a10d8ef220f Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <[email protected]> Gerrit-Reviewer: Alissa Bonas <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Arik Hadas <[email protected]> Gerrit-Reviewer: Daniel Erez <[email protected]> Gerrit-Reviewer: Liron Aravot <[email protected]> Gerrit-Reviewer: Maor Lipchuk <[email protected]> Gerrit-Reviewer: Omer Frenkel <[email protected]> Gerrit-Reviewer: Roy Golan <[email protected]> Gerrit-Reviewer: Tal Nisan <[email protected]> Gerrit-Reviewer: Vered Volansky <[email protected]> _______________________________________________ Engine-patches mailing list [email protected] http://lists.ovirt.org/mailman/listinfo/engine-patches
