[Codenarc-developer] codenarc support for @SuppressWarnings
Brought to you by:
chrismair
From: Chris M. <chr...@ea...> - 2010-11-05 02:00:21
|
Hamlet, This is really great! I had wondered about doing something like this, along the lines of what PMD does. But again, you took it beyond, and made it better, than what I would have likely ended up with. I like your use of the standard @SupressWarnings annotation. Chris -----Original Message----- From: Hamlet D'Arcy [mailto:ham...@gm...] Sent: Thursday, November 04, 2010 11:39 AM To: Cod...@li... Subject: [Codenarc-developer] codenarc support for @SuppressWarnings Hi chris, I just checked in DuplicateLiteral. The interesting part is that you can suppress the rule by annotating the class or method with: @SuppressWarning('DuplicateLiteral') First off, you might want to search the CodeNarc codebase for this suppression and see where I had to use it. You may want to clean up some code. Secondly, there is a general way to add SuppressWarning support to all rules by modifying the base visitor. I am going to take a look at doing this. What do you think? -- Hamlet D'Arcy ham...@gm... ---------------------------------------------------------------------------- -- The Next 800 Companies to Lead America's Growth: New Video Whitepaper David G. Thomson, author of the best-selling book "Blueprint to a Billion" shares his insights and actions to help propel your business during the next growth cycle. Listen Now! http://p.sf.net/sfu/SAP-dev2dev _______________________________________________ Codenarc-developer mailing list Cod...@li... https://lists.sourceforge.net/lists/listinfo/codenarc-developer |