|
Patch offers some refactoring of
DynamicURI to decrease a little bit the number of objects created via
toString()/getA().
- changed Vector to
ArrayList
- removed the boolean hasXXX in favor of
!XXX.isEmpty()
- allocate initially some room for the
StringBuffer rather than the default 16
- extracted renderQueryString and
renderPathInfo into a renderPairs method
- couple of javadoc fixes.
I also replaced URLEncoder by URIUtil from
commons-httpclient since URLEncoder is somewhat a performance killer (you
will need a more recent version than the one that update-jars grab. 2.0rc is
fine). A quick snapshot differencing via JProbe 4.0 beta reports that
URIUtil is twice faster and creates 20% less objects.
Use at your own discretion.
Testcase is of course not complete...but
hey, it's better than nothing :)
|
dynamicuri.patch
Description: Binary data
DynamicURITest.java
Description: Binary data
-- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
