Change in ovirt-engine[master]: core: AffinityRulesUtils lambda's left curly

Code Review gerrit at ovirt.org
Wed Mar 22 11:54:50 UTC 2017


>From Tal Nisan <tnisan at redhat.com>:

Tal Nisan has submitted this change and it was merged.

Change subject: core: AffinityRulesUtils lambda's left curly
......................................................................


core: AffinityRulesUtils lambda's left curly

Move the lambda's curly to the line the lambda is defined on, as per
the project's coding standards.

While the current checkstyle does not catch this violation, it will
fail with newer checkstyle versions that support Java 8 syntax.

Change-Id: I27edd31a2506d2535edf827e543d3d86d0c1f7e5
Related-To: https://bugzilla.redhat.com/1433408
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/scheduling/arem/AffinityRulesUtils.java
1 file changed, 1 insertion(+), 2 deletions(-)

Approvals:
  Tal Nisan: Looks good to me, approved
  Martin Sivák: Looks good to me, approved
  Jenkins CI: Passed CI tests
  Allon Mureinik: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I27edd31a2506d2535edf827e543d3d86d0c1f7e5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Martin Sivák <msivak at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>


More information about the Engine-commits mailing list