Wenyi Gao has posted comments on this change. Change subject: add a new tool for auto-generate python entities from schema ......................................................................
Patch Set 7: I would prefer that you didn't submit this (5 inline comments) .................................................... File src/codegen/genparams.py Line 3: # Copyright (c) 2010 Red Hat, Inc. Should be IBM. Line 138: return rc 'rc' won't defined if OSError happends. Line 146: def classReRule(cls): It would be better to keep the constant style. cls-->self .................................................... File src/codegen/paramsconf.py Line 1: # this configure is used to generate the params.py Add copyright information. Line 13: modules = ["from ovirtsdk.utils.reflectionhelper import ReflectionHelper"] Using capital to define global vars maybe more clear. -- To view, visit http://gerrit.ovirt.org/4880 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I916f63f1ac68ce2e31456a90c1291997f671af99 Gerrit-PatchSet: 7 Gerrit-Project: ovirt-engine-sdk Gerrit-Branch: master Gerrit-Owner: ShaoHe Feng <shao...@linux.vnet.ibm.com> Gerrit-Reviewer: Adam Litke <a...@us.ibm.com> Gerrit-Reviewer: Lei Li <li...@linux.vnet.ibm.com> Gerrit-Reviewer: Mark Wu <wu...@linux.vnet.ibm.com> Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com> Gerrit-Reviewer: ShaoHe Feng <shao...@linux.vnet.ibm.com> Gerrit-Reviewer: Shu Ming <shum...@linux.vnet.ibm.com> Gerrit-Reviewer: Wenyi Gao <we...@linux.vnet.ibm.com> Gerrit-Reviewer: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com> _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
