svn commit: r798054 - /tomcat/tc6.0.x/trunk/STATUS.txt

[rj...@apache.org]

Author: rjung
Date: Mon Jul 27 07:25:49 2009
New Revision: 798054

URL: http://svn.apache.org/viewvc?rev=798054&view=rev
Log:
Vote.

Modified:
    tomcat/tc6.0.x/trunk/STATUS.txt

Modified: tomcat/tc6.0.x/trunk/STATUS.txt
URL: http://svn.apache.org/viewvc/tomcat/tc6.0.x/trunk/STATUS.txt?rev=798054&r1=798053&r2=798054&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Mon Jul 27 07:25:49 2009
@@ -152,7 +152,7 @@
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=41661
   Thread safety issue in JspConfig.init() seen in production
   http://svn.apache.org/viewvc?rev=795210&view=rev
-  +1: markt, kkolinko
+  +1: markt, kkolinko, rjung
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=47413
@@ -160,33 +160,39 @@
   Without coercion, the concatenation of ((Object)null) + string results in
   "null" being printed.
   http://svn.apache.org/viewvc?rev=795466&view=rev
-  +1: kkolinko, markt
+  +1: kkolinko, markt, rjung
   -1: 
 
 * Fix the JDBC driver clean-up
   http://svn.apache.org/viewvc?view=rev&revision=789389
-  +1: markt
+  +1: markt, rjung
   -1: 
+  rjung: I don't really like the implicit 4KB class size limit.
+  I understand your rationale, but we don't know enough about other JVMs
+  and the future in general.
+  I think it would be nice to improve in a second step by resizing the
+  array according to the class size. AFAIK it's not a performance critical path.
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=46907
   Debug logging should not cause input stream to be swallowed
   http://svn.apache.org/viewvc?view=rev&revision=797168
-  +1: markt, kkolinko
+  +1: markt, kkolinko, rjung
   -1: 
 
 * https://issues.apache.org/bugzilla/show_bug.cgi?id=47566
   Update to NSIS 2.45
   http://svn.apache.org/viewvc?rev=797596&view=rev
-  +1: kkolinko
+  +1: kkolinko, rjung
   -1:
   kkolinko: Now it also requires the following patch, but I hope that
   sourceforge.net will fix their error:
     http://svn.apache.org/viewvc?rev=797600&view=rev
+  rjung: +1 for both as long as the problem lasts (SF-DL still seems to be broken).
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=41824
   Need to use canonical rather than binary form when writing code
   http://svn.apache.org/viewvc?rev=797607&view=rev
-  +1: markt
+  +1: markt, rjung
   -1: 
 
 * Improve build.properties comment in ant build files.



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org