ShaoHe Feng has posted comments on this change.

Change subject: add a new tool for auto-generate python entities from schema
......................................................................


Patch Set 5: (7 inline comments)

....................................................
Commit Message
Line 21: 
agree

....................................................
File src/codegen/paramsconf.py
Line 1: ######################!/usr/bin/python
surely, it can be removed.

Line 4: # this maybe useful, for some editor such as vi or eclips
sorry, Eclipse

Line 18: GENERATEDSSUPER_ATTRS = """
agree

Line 80:         "GeneratedsSuper": GENERATEDSSUPER_ATTRS}
agree

Line 136: # the function dict which will be take the same function in the 
params.py
surely.

Line 140: #       second,which class the functon belong to
a global function, it does not belong to any class.
different classes may have a same name functions, so the class name must be 
given in this case.

--
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: 5
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: 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: Zhou Zheng Sheng <zhshz...@linux.vnet.ibm.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to