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

--- Comment #1 from Mark Thomas <ma...@apache.org> ---
Regarding your points:

1. Clarity vs efficiency is always a trade off. It depends how much efficiency
has been lost. Do you have any performance numbers?

2. I don't see a test case either. I'd rather get any bugs in the decoder fixed
than put sticking plasters over other bits of code.

3. I'd prefer to do this first. That code is only still in the code base
because it support ByteChunk / CharChunk which should allow a more efficient
interface with the rest of the Tomcat code base. It is probably time that that
assumption is tested.

4. It is very unlikely we will be adding Commons Codec as a dependency to
Tomcat. We may do an svn copy and rename much like we have done for FileUpload
but the usefulness of that depends on the ByteChunk/CharChunk issues in 3. For
FileUpload I am looking at replacing the decoder with the JVM implementation.

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