[GitHub] [tomcat] martin-g commented on pull request #356: Micro optimization: replace "".equals(a) with a.isEmpty()

2020-09-12 Thread GitBox
martin-g commented on pull request #356: URL: https://github.com/apache/tomcat/pull/356#issuecomment-690909792 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [tomcat] martin-g commented on pull request #356: Micro optimization: replace "".equals(a) with a.isEmpty()

2020-09-12 Thread GitBox
martin-g commented on pull request #356: URL: https://github.com/apache/tomcat/pull/356#issuecomment-690909792 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [tomcat] martin-g commented on pull request #356: Micro optimization: replace "".equals(a) with a.isEmpty()

2020-09-11 Thread GitBox
martin-g commented on pull request #356: URL: https://github.com/apache/tomcat/pull/356#issuecomment-690909792 > It would be great just to fix all these problems at once I prefer to make one kind of change at a time. It is easier to detect regressions and impact this way.