by the way, where we really need this is on the trunk, not the branch. Egon Willighagen wrote:
>Bob, > >as promised, I've installed PMD 4.1 in the v11_4/ branch. > >(BTW, I forgot that PMD has been running on Jmol code for a long, long >time... for some years at least, which is why I forgot about it :) >See http://pmd.sourceforge.net/scoreboard.html) > >You can run it with: > >$ ant -f build-pmd.xml > >Which will create a HTML file: > >build/pmd/jmol.html > >Currently, 47 unused bits of code have been found. If you feel some >code is fine as in, and you don't want the PMD report about it, just >add '// NOPMD' to the end of the line marked by PMD as having an >error. > >Right now, only unused code is detected. You can edit the >pmd.config.xml and add other 'rulesets'. > >I can recommend to add: > >basic >junit >optimizations >imports > >But there are many interesting others... > >If you hate one or two rules in a certain set, you can tune things >with the configuration file. I have been doing that for the CDK for a >long time now. See >http://cdk.svn.sourceforge.net/viewvc/cdk/trunk/cdk/pmd/ for examples. > >Egon > > > -- Robert M. Hanson Professor of Chemistry St. Olaf College Northfield, MN http://www.stolaf.edu/people/hansonr If nature does not answer first what we want, it is better to take what answer we get. -- Josiah Willard Gibbs, Lecture XXX, Monday, February 5, 1900 ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2005. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ Jmol-developers mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/jmol-developers
