[GitHub] [tomcat] markt-asf commented on issue #222: EmbeddedServletOptions constructor has been refactored

[GitBox <gi...@apache.org>]

markt-asf commented on issue #222: EmbeddedServletOptions constructor has been refactored
URL: https://github.com/apache/tomcat/pull/222#issuecomment-550298013
 
 
   This appears to be a purely cosmetic change.
   Such changes can make back-porting more difficult unless the change is back-ported to all currently supported versions.
   The change makes this class inconsistent with similar code throughout the Tomcat code base.
   Changes such as this carry the risk of introducing regressions.
   Unless a good reason can be provided for applying this, I am leaning towards closing this PR.

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org