Hi,

On Fri, Mar 14, 2014 at 11:51 AM, Felix Meschberger <fmesc...@adobe.com> wrote:
> ...I have the impression using a regular expression to replace a single
> character with another character is overkill and replacing this code
> with a simple "String.replace(':', '/')" might be more appropriate....

If that's covered by tests, and those tests handle the case where
there are several colons in a resource type (which is legal, right?)
and your measurements show significantly better performance, why not
;-)

-Bertrand

Reply via email to