Hi,
Here is a minor but annoying issue ;)
I've caught a couple of commits lately that uses a non-eclipse standard for the naming
of i18n messages.
The commits used:
public static String ERROR_RUNTIME_NAME;
But the i18n standard used in Eclipse and (99% of) JBoss Tools is:
public static String errorRuntimeName;
I thought I caught them all until Grid told me SeamUIMessages has the same issue,
which we should get fixed too at some point (and there might be more ;)
So just thought I would make it sure it's known so when you i18n other new things
going forward -
please use the Eclipse standard.
p.s. the reason for using camelcase here vs uppercase is that uppercase is normally
used for constants - this field is actually not a final constant (its initialized
dependent on
users settings) AND these i18n messages is used *all* over so having them all being
uppercase makes
it harder to read the code.............but it mostly comes down to let us use the same
mechanism/standard
as Eclipse to make things more uniform.
Sorry for the interruption ;)
/max
> But the i18n standard used in Eclipse and (99% of) JBoss Tools
is:
>
> public static String errorRuntimeName;
>
correction, it's actually:
> public static String ErrorRuntimeName;
>
or
> public static String Error_Runtime_Name;
>
Take your pick :)
/max
_______________________________________________
jbosstools-dev mailing list
jbosstools-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/jbosstools-dev