Change in ovirt-engine-sdk-java[master]: codegen: Sort class members alphabetically

juan.hernandez at redhat.com juan.hernandez at redhat.com
Fri Feb 13 15:39:45 UTC 2015


Juan Hernandez has submitted this change and it was merged.

Change subject: codegen: Sort class members alphabetically
......................................................................


codegen: Sort class members alphabetically

Currently the code generator doesn't sort the class members in any
particular order. This means that each time the code is re-generated
things will potentially be in a different order, making it difficult to
visually analyze the changes (with the "diff" tool, for example). To
avoid this issue this patch changes the generator so that class members
will be sorted alphabetically.

Change-Id: I59940162b7fad175bd0bce7585f43cac65acb78a
Signed-off-by: Juan Hernandez <juan.hernandez at redhat.com>
---
M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/holders/AbstractCollectionHolder.java
M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/holders/ResourceHolder.java
M ovirt-engine-sdk-java-codegen/src/main/java/org/ovirt/engine/sdk/codegen/rsdl/ApiCodegen.java
3 files changed, 75 insertions(+), 43 deletions(-)

Approvals:
  Martin Peřina: Looks good to me, but someone else must approve
  Juan Hernandez: Verified; Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I59940162b7fad175bd0bce7585f43cac65acb78a
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine-sdk-java
Gerrit-Branch: master
Gerrit-Owner: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: automation at ovirt.org



More information about the Engine-commits mailing list