Allon Mureinik has posted comments on this change.

Change subject: core: Remove unused action type enums.
......................................................................


Patch Set 1: Looks good to me, approved

(2 inline comments)

I'd squash removing the params classes in to this patch, but I guess it doesn't 
really matter.

....................................................
Commit Message
Line 3: AuthorDate: 2013-02-11 18:42:34 +0200
Line 4: Commit:     Maor Lipchuk <[email protected]>
Line 5: CommitDate: 2013-02-11 18:42:34 +0200
Line 6: 
Line 7: core: Remove unused action type enums.
perhaps "enum valiues" instead of "enums"?
Line 8: 
Line 9: The removed enums does not have any references, and could be
Line 10: deleted.
Line 11: 


Line 5: CommitDate: 2013-02-11 18:42:34 +0200
Line 6: 
Line 7: core: Remove unused action type enums.
Line 8: 
Line 9: The removed enums does not have any references, and could be
s/does/do/
Line 10: deleted.
Line 11: 
Line 12: Change-Id: I8b837f7ba89ba3d78968c9b0b308405f6d2d2dbe


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8b837f7ba89ba3d78968c9b0b308405f6d2d2dbe
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Allon Mureinik <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Daniel Erez <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Maor Lipchuk <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Tal Nisan <[email protected]>
Gerrit-Reviewer: ofri masad <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to