On 29/05/2019 08:46, r...@apache.org wrote: > This is an automated email from the ASF dual-hosted git repository. > > remm pushed a commit to branch master > in repository https://gitbox.apache.org/repos/asf/tomcat.git > > > The following commit(s) were added to refs/heads/master by this push: > new b510524 Move ADD_CHILD_EVENT to before the optional container start > b510524 is described below > > commit b51052483ec3687c6b4db05288d7546d8431eaf0 > Author: remm <r...@apache.org> > AuthorDate: Wed May 29 09:45:54 2019 +0200 > > Move ADD_CHILD_EVENT to before the optional container start > > Since child start is optional and may be delayed, container listeners > already had to handle this situation (and the mapper indeed does). As > the (correct) sequence is now add to collection -> add_child_event -> > start, on stop it is updated to stop -> remove_child_event -> remove > from collection (for consistency, in practice it is unlikely to make any > difference. > Revert the addition of two container events. > Thanks Mark for the review and the comment.
Thanks for making those changes. Mark --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org