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.

So, at the very least, I think that getGuiName should be documented a
little more fully.

-- 
Angus

Reply via email to