Change in ovirt-engine[master]: findbugs: Clean up Dubious Method warnings
emesika at redhat.com
emesika at redhat.com
Tue Apr 8 09:16:17 UTC 2014
Eli Mesika has submitted this change and it was merged.
Change subject: findbugs: Clean up Dubious Method warnings
......................................................................
findbugs: Clean up Dubious Method warnings
Findbugs' Dubious Method Used warning are about calling methods that
would work, but perform the required task inefficiently (e.g., calling
new Boolean(true) instead of Boolean.valueOf(true), or even
Boolean.TRUE).
This patch removes the all-out filter for this category of warnings.
Issues previously suppressed by this filter are either fixed where
possible, or replaced with a more fine-grained filter where not
possible.
A notable case where these issues cannot be fixed in is GWT's
auto-generated hashCode() methods for classes containing a boolean
member.
Change-Id: I6b904dfea1991b6ce5bb7c5a534127ee10e56a24
Signed-off-by: Allon Mureinik <amureini at redhat.com>
---
M backend/manager/modules/restapi/jaxrs/src/main/java/org/ovirt/engine/api/restapi/resource/BaseBackendResource.java
M backend/manager/modules/restapi/types/src/main/java/org/ovirt/engine/api/restapi/types/HostNicMapper.java
M exclude-filters-general.xml
M frontend/webadmin/modules/frontend/exclude-filters.xml
M frontend/webadmin/modules/gwt-common/exclude-filters.xml
M frontend/webadmin/modules/gwt-common/src/main/java/org/ovirt/engine/ui/common/widget/renderer/VolumeTransportTypeRenderer.java
M frontend/webadmin/modules/webadmin/exclude-filters.xml
7 files changed, 15 insertions(+), 8 deletions(-)
Approvals:
Tal Nisan: Looks good to me, approved
Juan Hernandez: Looks good to me, but someone else must approve
Allon Mureinik: Verified
--
To view, visit http://gerrit.ovirt.org/26473
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I6b904dfea1991b6ce5bb7c5a534127ee10e56a24
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Alissa Bonas <abonas at redhat.com>
Gerrit-Reviewer: Allon Mureinik <amureini at redhat.com>
Gerrit-Reviewer: Daniel Erez <derez at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Juan Hernandez <juan.hernandez at redhat.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: Vojtech Szocs <vszocs at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: automation at ovirt.org
Gerrit-Reviewer: oVirt Jenkins CI Server
More information about the Engine-commits
mailing list