Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [jwt-dev] Important : NLS messages

Hi all,

Just a special case for EMF generated classes that contains not-externalized strings: If you see that the class is generated (ie comment on the class contains @generated), then simply add the annotation @SuppressWarnings("nls") on that class.

Indeed, we cannot spend so much time on generated class to manually externalize them or to add NON-NLS, all the more that EMF strings have a logical meaning.

I'll add this special case in the wiki entry about externalization.

Regards,
Mickael

Mickael Istria a écrit :
Dear contributors,

Using NLS internationalized and externalized strings is a really important requirement for Galileo. So, I urge all of you to mark non externalized strings as warnings (or errors) while you are developing. That way, you'll be easily able to see which strings are externalized and which are not.

I attached a screenshot that shows where to activate these warnings. Here are some other resources on this topic if you're interested: * http://www.lunar-ocean.com/eclipse-committers-turn-your-nls-warnings-on/
* https://bugs.eclipse.org/bugs/show_bug.cgi?id=261766
* http://help.eclipse.org/stable/index.jsp?topic=/org.eclipse.platform.doc.isv/reference/misc/message_bundles.html
* ...

Concretely, here are some good practices for NLS:
* If your string is related to UI, then it *must* be externalized and referenced from a NLS Message Bundle * If your string is a "logic" string (for examples if it the name of a plugin extension point attribute, or "true"/"false" strings) and if it is used several times, then create a constant for that string and use it. * It your string is _never_ intended to be displayed (for example log messages or bundle names...) then, add a //NON-NLS-n comment.

I've already externalized or NON-NLS-ed most of strings in we and transformations plugins. However, some of them probably still remain. So please take care of it in your next developments, because it is quite time-consuming to do it in after development on existing code.

Regards,
Mickael (stressed by bug 261766 ;)


------------------------------------------------------------------------

------------------------------------------------------------------------

_______________________________________________
jwt-dev mailing list
jwt-dev@xxxxxxxxxxx
https://dev.eclipse.org/mailman/listinfo/jwt-dev



Back to the top