[ 
http://www.stripesframework.org/jira/browse/STS-694?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=11796#action_11796
 ] 

Jon Stevens commented on STS-694:
---------------------------------

Is it that bad to add a @SuppressWarnings("unused") to those methods?

The benefit of having a clean warning free build is that it is easier to 
quickly see problems if they crop up because the icons show you in the Package 
Explorer.

Turning off that error in Eclipse preferences could unfortunately allow 
mistakes to slip through.

I know it isn't an ideal solution to add @SW, but honestly, it is only a few 
places and they are just test classes.


> patch to remove warnings in eclipse 3.5
> ---------------------------------------
>
>                 Key: STS-694
>                 URL: http://www.stripesframework.org/jira/browse/STS-694
>             Project: Stripes
>          Issue Type: Improvement
>         Environment: using eclipse 3.5 and svn trunk
>            Reporter: Jon Stevens
>            Priority: Trivial
>             Fix For: Release 1.6
>
>         Attachments: patch.txt
>
>
> I tend to like a build without any warnings. I just checked out the source 
> code to stripes to check it out and found a bunch of areas where eclipse was 
> throwing warnings. This patch just adds a bunch of <?> to the end of various 
> places where eclipse 3.5 was complaining. I also use Eclipse Java->Save 
> Actions to improve the look and feel of the source code without having to do 
> any work. This created a slightly larger patch file. Here are my settings:
> Add 'this' qualifier to unqualified field accesses
> Add 'this' qualifier to unqualified method accesses
> Convert for loops to enhanced for loops
> Add paranoiac parentheses
> Remove unused imports
> Add missing '@Override' annotations
> Add missing '@Deprecated' annotations
> Remove unnecessary casts
> Remove unnecessary '$NON-NLS$' tags
> Remove trailing white spaces on all lines

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://www.stripesframework.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

------------------------------------------------------------------------------
Come build with us! The BlackBerry&reg; Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9&#45;12, 2009. Register now&#33;
http://p.sf.net/sfu/devconf
_______________________________________________
Stripes-development mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/stripes-development

Reply via email to