Angus Leeming <[EMAIL PROTECTED]> writes:
| Jean-Marc Lasgouttes wrote:
>
>> But I guess you know by now that adding random transformation (like
>> trim()) in helper functions just because you are lazy is not always a
>> good idea ;)
>
| Of course, getGuiName(...) suggests a transformation whereas getString
| doesn't imply a trim. getTrimmedString, however, would be perfectly
| reasonable and not at all lazy.
And getGuiName should never be used for any comparisons... just to be
displayed. So imo the translation should be done there...
-... but... if the change to make that happen is "largish" then I'd go
with this proposed change and try to fix it proper in 1.5.
--
Lgb
