Howdy, PMD is a good tool. There's a large, and increasing, overlap between it and Checkstyle. Both of them IMHO can be misleading and shouldn't be trusted blindly. For example, both (Checkstyle only in its latest version) will complain about non-final parameters to methods, method "not designed for extension," etc. For an open-source framework designed to be extensible like log4j, these are non issues and the checks should be turned off in PMD/Checkstyle. We should agree to a common Checkstyle/PMD checklist (unfortunately they have different formats) before making large sweeping changes based on PMD/Checkstyle output.
(Of course this doesn't apply to trivial things like missing javadoc comments) Yoav Shapira Millennium ChemInformatics >-----Original Message----- >From: Berin Loritsch [mailto:[EMAIL PROTECTED] >Sent: Wednesday, June 25, 2003 5:25 PM >To: Log4J Developers List >Subject: Re: PMD > >Ceki Gülcü wrote: > >> At 02:15 PM 6/25/2003 -0700, you wrote: >> >>> Sorry, http://pmd.sourceforge.net/ >> >> >> LOL. I thought PMD was an acronym and "Packet Mode Data", "Physical >> Medium Dependent", "Polarization Mode Dispersion" sounded out of place. > > >:D My favorite backronym is "Protein Mutant Database" from their own > page (http://pmd.sourceforge.net/meaning.html). > >-- > >"They that give up essential liberty to obtain a little temporary safety > deserve neither liberty nor safety." > - Benjamin Franklin > > >--------------------------------------------------------------------- >To unsubscribe, e-mail: [EMAIL PROTECTED] >For additional commands, e-mail: [EMAIL PROTECTED] This e-mail, including any attachments, is a confidential business communication, and may contain information that is confidential, proprietary and/or privileged. This e-mail is intended only for the individual(s) to whom it is addressed, and may not be saved, copied, printed, disclosed or used by anyone else. If you are not the(an) intended recipient, please immediately delete this e-mail from your computer system and notify the sender. Thank you. --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
