markt-asf commented on PR #652:
URL: https://github.com/apache/tomcat/pull/652#issuecomment-1706738930

   We have started to get a lot of PRs removing single instances of unnecessary 
parentheses. While we are generally in favour of this sort of clean-up, one PR 
per instance is not scaleable and will generate a LOT of noise on the mailing 
list.
   
   Automated tools aren't the answer as there are a reasonable proportion of 
these unnecessary parentheses we will want to keep as they aid readability of 
the code.
   
   If you expanded this PR to cover all the classes in the package then that is 
probably at a scale that would be manageable.


-- 
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

Reply via email to