https://issues.apache.org/bugzilla/show_bug.cgi?id=49234

--- Comment #76 from Mark Thomas <ma...@apache.org> 2010-08-11 12:59:22 EDT ---
Thanks for the updated patch.

I wasn't thinking of explicitly setting startChildren in the addChild() method
but it is a good idea and I like it. However, you need to rework your patch to
ensure startChildren is always reset, even if an Exception occurs.

Also, there is no need to manipulate startChildren in startChild() - at least I
can't see one.

-- 
Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

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

Reply via email to