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