DaanHoogland commented on issue #3275: [CLOUDSTACK-10408] Fix 
String.replaceAll() to replace() for better performance
URL: https://github.com/apache/cloudstack/pull/3275#issuecomment-579806955
 
 
   @rhtyd the characters %20'-/" and the names of bridges are used in this PR, 
I don't see the regexp risk spoken about in that link. I have no objections to 
this

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to