Bug 263722 - [nls tooling] Externalize Strings wizard: option to add @SuppressWarnings("nls") annotation
Summary: [nls tooling] Externalize Strings wizard: option to add @SuppressWarnings("nl...
Status: ASSIGNED
Alias: None
Product: JDT
Classification: Eclipse Project
Component: Text (show other bugs)
Version: 3.4   Edit
Hardware: All All
: P3 enhancement with 1 vote (vote)
Target Milestone: ---   Edit
Assignee: JDT-Text-Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-02-04 19:50 EST by Sean Flanigan CLA
Modified: 2009-06-25 11:06 EDT (History)
2 users (show)

See Also:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Sean Flanigan CLA 2009-02-04 19:50:23 EST 
Build ID: I20080617-2000

A source file with many non-externalisable strings often ends up with a lot of NON-NLS comments, which can be untidy.  This tends to discourage the use of the compiler option which generates warnings/errors about non-externalised strings.

What would help is an extra button on the Externalize String wizard, perhaps labelled "Ignore all (annotation)" or "Always ignore in this file", which adds the annotation @SuppressWarnings("nls") to the entire class.

A similar quick-fix would be very handy too.

Some background, and original suggestion by Christian W. Damus: https://bugs.eclipse.org/bugs/show_bug.cgi?id=261766#c7
Comment 1 Sean Flanigan CLA 2009-06-04 00:25:01 EDT 
A related annoyance: the Externalize String wizard doesn't respect @SuppressWarnings("nls") - it still wants to externalize strings in classes/members which are marked with the annotation.
Comment 2 Olivier Thomann CLA 2009-06-25 10:58:48 EDT 
Moving to JDT/UI
Comment 3 Dani Megert CLA 2009-06-25 11:03:16 EDT 
>A related annoyance: the Externalize String wizard doesn't respect
>@SuppressWarnings("nls") 
See bug 218392.