Findbugs issues

David Caro dcaroest at redhat.com
Wed Apr 23 16:10:03 UTC 2014


Hi!

There are still some findbugs issues merged on ovirt-engine master, and I've
tracked them down to a couple commits of yours, can you please review them and
if they were introduced there, fix them?

Thanks!!

Commit 4c2d171a701c04f5b3a82690826aa72b08f2f3e1 by Martin Perina
HostGeneralModel.java:641, ES_COMPARING_PARAMETER_STRING_WITH_EQ, Priority: Low

Comparison of String parameter using == or != in
org.ovirt.engine.ui.uicommonweb.models.hosts.HostGeneralModel.setKdumpStatus(String)

This code compares a java.lang.String parameter for reference equality using the
== or != operators. Requiring callers to pass only String constants or interned
strings to a method is unnecessarily fragile, and rarely leads to measurable
performance gains. Consider using the equals(Object) method instead.


Commit 749eba95219f36c0ad5ee8216b65a8efc8a1bae4 by Tomas Jelinek
VmListModel.java:424, URF_UNREAD_FIELD, Priority: Low

Unread field:
org.ovirt.engine.ui.uicommonweb.models.vms.VmListModel.privateCustomPropertiesKeysList

This field is never read.  Consider removing it from the class.


pd. We have improved infra enough and we are gonna enable back the per-patch
checks to avoid this from happening in the future so we will not bother you
anymore with these issues :)

-- 
David Caro

Red Hat S.L.
Continuous Integration Engineer - EMEA ENG Virtualization R&D

Email: dcaro at redhat.com
Web: www.redhat.com
RHT Global #: 82-62605

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: OpenPGP digital signature
URL: <http://lists.ovirt.org/pipermail/infra/attachments/20140423/b0138735/attachment.sig>


More information about the Infra mailing list