https://bz.apache.org/bugzilla/show_bug.cgi?id=59219

--- Comment #8 from Mark Thomas <ma...@apache.org> ---
Thanks for the patch. It is being looked at. Currently I'm seeing odd behaviour
on Windows so I want to test it on another platform.

I do have some basic feedback at this point. Expect some more when once I have
test test working:
1. Use separate methods for separate tests. It makes it easier to determine
which tests are failing.
2. We aim to keep the code warning free. If you use Eclipse, checkout the
configuration settings we use under res/ide-support/eclipse
3. It looks like you should be able to re-use the existing tracking listener.

-- 
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