Laszlo Hornyak has submitted this change and it was merged. Change subject: core: (Partially) Cleanup RunVmCommand ...................................................................... core: (Partially) Cleanup RunVmCommand (Partial) java cleanup to RunVmCommand: 1. Replaced deprecated StringHelper with common's StringUtils. 2. Replaced singleton access to external resources with protected access methods where possible. 3. Removed redundent null checks on canDoMessages. 4. Removed commented out block. 5. Made the log final. 6. Eclipse IDE formatting. Change-Id: I1674446fbd448d800824cfc29c960a90329f31fa Signed-off-by: Allon Mureinik <amureini@redhat.com> --- M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/RunVmCommand.java 1 file changed, 54 insertions(+), 80 deletions(-) Approvals: Laszlo Hornyak: Verified; Looks good to me, approved -- To view, visit http://gerrit.ovirt.org/5829 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1674446fbd448d800824cfc29c960a90329f31fa Gerrit-PatchSet: 3 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Allon Mureinik <amureini@redhat.com> Gerrit-Reviewer: Daniel Erez <derez@redhat.com> Gerrit-Reviewer: Eli Mesika <emesika@redhat.com> Gerrit-Reviewer: Laszlo Hornyak <lhornyak@redhat.com> Gerrit-Reviewer: Maor Lipchuk <mlipchuk@redhat.com>