Change in ovirt-engine[master]: engine: Fix for RHBZ#968958 - Refactor unnecessary while loo...

yzaslavs at redhat.com yzaslavs at redhat.com
Thu Aug 1 14:13:18 UTC 2013


Yair Zaslavsky has submitted this change and it was merged.

Change subject: engine: Fix for RHBZ#968958 - Refactor unnecessary while loop
......................................................................


engine: Fix for RHBZ#968958 - Refactor unnecessary while loop

Fixing unnecessary while loop in JndiAction class.
Under any circumstances the while loop runs only once,
hence it's not necessary and simple "if" statement can do the same.

Change-Id: I938726d631ac4ccb91cc68791e8e349dd43895ea
Bug-Url: https://bugzilla.redhat.com/968958
Signed-off-by: Tomas Dosek <tdosek at redhat.com>
---
M backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/kerberos/JndiAction.java
1 file changed, 7 insertions(+), 10 deletions(-)

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


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

Gerrit-MessageType: merged
Gerrit-Change-Id: I938726d631ac4ccb91cc68791e8e349dd43895ea
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomáš Došek <tdosek at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Tomáš Došek <tdosek at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list