Change in ovirt-engine[master]: frontend: Remove IteratorUtils class

tnisan at redhat.com tnisan at redhat.com
Mon Feb 17 14:06:18 UTC 2014


Tal Nisan has submitted this change and it was merged.

Change subject: frontend: Remove IteratorUtils class
......................................................................


frontend: Remove IteratorUtils class

Since we changed the items held in the ListModel from Iterable to
Collection, we can now remove IteratorUtils as it's redundant, instead of
iterating on the Iterable we can just use the size() method that Collection
object has

Change-Id: Iace61d708d6e66da4b898a6d6069136e1c838c4f
Signed-off-by: Tal Nisan <tnisan at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/models/SearchableListModel.java
D frontend/webadmin/modules/uicompat/src/main/java/org/ovirt/engine/ui/uicompat/IteratorUtils.java
2 files changed, 2 insertions(+), 42 deletions(-)

Approvals:
  Tal Nisan: Verified
  Vojtech Szocs: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Iace61d708d6e66da4b898a6d6069136e1c838c4f
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server



More information about the Engine-commits mailing list