https://bz.apache.org/bugzilla/show_bug.cgi?id=62757
--- Comment #7 from Mark Thomas <ma...@apache.org> --- Created attachment 36186 --> https://bz.apache.org/bugzilla/attachment.cgi?id=36186&action=edit Alternative patch Thanks for the additional explanation. I agree with all of it. I had similar concerns when I was reviewing your original report. An additional concern was that the code seemed to be getting more fragile. In particular, firstTime and theServlet == null seemed to be doing the same thing (and ditto for tagHandlerClass). In an effort to address this, I have worked up an alternative patch that removes firstTime. I have attached it to this report. Feedback on that patch would be appreciated - particularly if you are able to test it in your environment which is likely to be more complex than the simple tests I have been doing with the example app. -- 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
