Thanks for the heads-up... I didn't look to see what the complaints were actually flagging... your right, rules like that definitely shouldn't be enabled if that's the kind of thing it is going to catch... and that one doesn't look configurable enough to get rid of those, and I certainly don't want to just "ignore" close to 10,000 warnings :)
-- Frank W. Zammetti Founder and Chief Software Architect Omnytex Technologies http://www.omnytex.com On Tue, August 9, 2005 9:43 am, [EMAIL PROTECTED] said: > Frank W. Zammetti said: >> * StrictDuplicateCode >> Duplicate code is just plain bad... not in terms of something not >> working, although that is certainly possible in some >> situations, but >> it's just a sign of carelessness. If nothing else, I'm sure no one >> wants the Struts code base to show any carelessness that >> could easily >> be avoided. >> [snip] >> >> The second is the updated version with all the above checks added... >> this results in 16584 complaints, which is clearly a lot... however, > the >> vast majority of the additional complaints result from the >> StrictDuplicateCode and are probably ignorable, so that's one rule > that >> on second thought maybe shouldn't be added... > > I would say this rule too simpleminded to be usable. I took a quick > look at the output, and it seems that it's complaining about the same > license boilerplate at the top of every file. No wonder it found so > many complaints! > --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
