Egon, I don't have any time for this now, but feel free to work with it yourself. Lots of code should appear to be unused because that's how we compartmentalize the JAR files. All classes outside the core are accessed with classForName().
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
