Laszlo Hornyak has posted comments on this change.

Change subject: [WIP] core: Introducing findbugs filter to bll module
......................................................................


Patch Set 3: (1 inline comment)

some questions inline.

....................................................
File backend/manager/modules/bll/exclude-filters.xml
Line 8:      <Match>
Line 9:        <Class name="org.ovirt.engine.core.bll.RunVmCommandBase" />
Line 10:        <Method name="delay"/>
Line 11:        <Bug code="RV"/>
Line 12:      </Match>
We will have to put all the nasty code here to be excluded from the report? How 
easy will that be to maintain?
Doesn't that also mean that we will put it here rather than fixing it?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I26b9cafc1832022693a7386277972a7e48a53f0d
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Allon Mureinik <amure...@redhat.com>
Gerrit-Reviewer: Laszlo Hornyak <lhorn...@redhat.com>
Gerrit-Reviewer: Michael Pasternak <mpast...@redhat.com>
Gerrit-Reviewer: Mike Kolesnik <mkole...@redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to