markt-asf commented on PR #651: URL: https://github.com/apache/tomcat/pull/651#issuecomment-1706340615
The switch to using a wildcard import would need to be reverted as would the changes to blank lines between methods. The null check is necessary in some form and it is more concise than the alternative proposed in this PR. The switch to using an array list has merit. I'll implement that change shortly. -- 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. To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org