Change in ovirt-engine[master]: frontend: firstOrDefault should call overloading method

tnisan at redhat.com tnisan at redhat.com
Sun Nov 22 10:33:02 UTC 2015


Tal Nisan has submitted this change and it was merged.

Change subject: frontend: firstOrDefault should call overloading method
......................................................................


frontend: firstOrDefault should call overloading method

This patch adds TruePredicate so that firstOrDefault(source) can call its
overloading method, firstOrDefault(source, predicate), instead of
duplicating the same code.
In addition, it makes firstOrDefault(source, predicate) null safe.

Change-Id: Ia8dfdf8208b3c16f244d03f6a81b478de3e4758f
Related-To: https://bugzilla.redhat.com/1282085
Signed-off-by: Idan Shaby <ishaby at redhat.com>
---
M frontend/webadmin/modules/uicommonweb/src/main/java/org/ovirt/engine/ui/uicommonweb/Linq.java
1 file changed, 13 insertions(+), 9 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved; Passed CI tests
  Allon Mureinik: Looks good to me, but someone else must approve
  Idan Shaby: Verified



-- 
To view, visit https://gerrit.ovirt.org/48640
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia8dfdf8208b3c16f244d03f6a81b478de3e4758f
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Idan Shaby <ishaby at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list