Roy Golan has submitted this change and it was merged. Change subject: core: cleanup in CommandBase ...................................................................... core: cleanup in CommandBase 1. Move field declarations up to the beginning of the class 2. Change the style of comments on fields to more compact style 3. Rename isReleaseExecute field to releaseLocksAtEndOfExecute to conform the java standard and be more expressive Change-Id: I551e00b86f18b62f95828b14f5fe54280eea6bb0 Signed-off-by: Arik Hadas <ahadas@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java 1 file changed, 16 insertions(+), 26 deletions(-) Approvals: Roy Golan: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/15886 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I551e00b86f18b62f95828b14f5fe54280eea6bb0 Gerrit-PatchSet: 2 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Arik Hadas <ahadas@redhat.com> Gerrit-Reviewer: Liron Ar <laravot@redhat.com> Gerrit-Reviewer: Omer Frenkel <ofrenkel@redhat.com> Gerrit-Reviewer: Roy Golan <rgolan@redhat.com> Gerrit-Reviewer: Yair Zaslavsky <yzaslavs@redhat.com> Gerrit-Reviewer: oVirt Jenkins CI Server