Change in ovirt-engine[master]: engineencryptutils: cleanup

yzaslavs at redhat.com yzaslavs at redhat.com
Wed Jan 30 08:21:38 UTC 2013


Yair Zaslavsky has submitted this change and it was merged.

Change subject: engineencryptutils: cleanup
......................................................................


engineencryptutils: cleanup

Remove dead code.
Remove redundant variables.
Rework to reach somewhat similar structure between functions.
Remove the use of StringHelper.

Not that the result is good... but at least something in the right way.

Change-Id: I9c1004e42ee92347cbf97011e3216a1ac5c23173
Signed-off-by: Alon Bar-Lev <alonbl at redhat.com>
---
M backend/manager/modules/engineencryptutils/pom.xml
M backend/manager/modules/engineencryptutils/src/main/java/org/ovirt/engine/core/engineencryptutils/EncryptionUtils.java
2 files changed, 20 insertions(+), 114 deletions(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I9c1004e42ee92347cbf97011e3216a1ac5c23173
Gerrit-PatchSet: 6
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>



More information about the Engine-commits mailing list