FindBugsHowto

Revision as of 16:20, 11 December 2012 by Phejl (Talk | contribs)

How to use FindBugs with NetBeans sources

This document describes how FindBugs is used during the development of NetBeans.

Running FindBugs locally

For every NetBeans module, you can run the findbugs check by typing:

cd your.module
ant findbugs
This will produce the findbugs report file in
nbbuild/build/findbugs/org-netbeans-modules-your-module.xml
and fails if at least one warning was discovered. No file is created if there is no problem in the tested module.


There is also a target in nbbuild to execute the test for all modules in a cluster at once:

ant -f nbbuild/build.xml findbugs-cluster
This produces the output files for each module with a problem in
nbbuild/build/findbugs/
directory.

Evaluating FindBugs warnings

The report containing FindBugs warnings can be viewed using the FindBugs UI. This tool can be downloaded from the FindBugs website: http://findbugs.sourceforge.net/. After installing FindBugs, you can run the tool using the
findbugs[.bat]
script in the bin directory. In the FindBugs UI, you may want to sort the issues by priority, by dragging the labels at the top left.

Suppressing warnings

In some cases, you may want to suppress a particular warning, rather than fix it. Always think twice about suppressing as FindBugs is usually right. If you really know it is a false warning you can do the following:

  1. Make sure your module uses the Common Annotations module (api.annotations.common).
  2. Find the id of the warning you want to suppress. Open the FindBugs report in a text editor, and find the <BugInstance> element for your warning. The value of the type attribute of this element is your warning id. The warning id is also displayed in FindBugs UI:
  3. Add the following annotation to the code (method, class, instance variable, local variable etc.) in which you want to suppress the warning:
    @org.netbeans.api.annotations.common.SuppressWarnings("your_warning_id")
    . There is an API enhancement [1] which adds a justification attribute to the annotation. Once in trunk the code might look like this:
    @org.netbeans.api.annotations.common.SuppressWarnings("your_warning_id", justification="Explanation why this is false warning")
    . Until then use following pattern:
    @org.netbeans.api.annotations.common.SuppressWarnings("your_warning_id") // justification="Explanation why this is false warning"
  4. It is good to be consistent when justifying the same type of things.

Additional information

Not logged in. Log in, Register

By use of this website, you agree to the NetBeans Policies and Terms of Use. © 2012, Oracle Corporation and/or its affiliates. Sponsored by Oracle logo