Change in ovirt-engine[master]: frontend: Improve DontPrune aspect code

[Code Review]

>From Vojtech Szocs <vszocs at redhat.com>:

Vojtech Szocs has submitted this change and it was merged.

Change subject: frontend: Improve DontPrune aspect code
......................................................................


frontend: Improve DontPrune aspect code

- added some comments to explain the details
- use Java 8 stream API to reduce code complexity

Note: in aop.xml file, we're configuring AOP weaver to apply
aspects not only to the code we want to advise (3rd party GWT
class ControlFlowAnalyzer) but also to the aspect Java class
itself (DontPrune). Without this, the build fails, so I guess
this is needed simply to ensure the aspect's Java class gets
properly loaded by AspectJ AOP weaver.

Change-Id: I14e84d3193cfcc49582d3cf85e5140be8e364eb0
Signed-off-by: Vojtech Szocs <vszocs at redhat.com>
---
M frontend/webadmin/modules/gwt-aop/src/main/java/org/ovirt/engine/ui/gwtaop/DontPrune.java
M frontend/webadmin/modules/gwt-aop/src/main/resources/META-INF/aop.xml
2 files changed, 31 insertions(+), 29 deletions(-)

Approvals:
  Juan Hernandez: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Allon Mureinik: Looks good to me, but someone else must approve
  Greg Sheremeta: Looks good to me, approved
  Vojtech Szocs: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I14e84d3193cfcc49582d3cf85e5140be8e364eb0
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: Alexander Wels <awels at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Greg Sheremeta <gshereme at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Scott Dickerson <sdickers at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>