Author: remm
Date: Fri Jul 11 07:17:25 2008
New Revision: 675960

URL: http://svn.apache.org/viewvc?rev=675960&view=rev
Log:
- Votes.

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=675960&r1=675959&r2=675960&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Jul 11 07:17:25 2008
@@ -52,20 +52,20 @@
   Don't assume UTF-8 and use the correct encoding when generating
   tomcat-users.xml from the Windows installer.
   http://svn.apache.org/viewvc?rev=674335&view=rev
-  +1: markt, fhanik, jfclere
+  +1: markt, fhanik, jfclere, remm
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=45285
   Need to look for annotations in class hierarchy
   http://people.apache.org/~markt/patches/2008-07-09-bug45285.patch
-  +1: markt, jfclere
+  +1: markt, jfclere, remm
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=42678
   Only ignore docBase it it really is a subdir of appBase
   Patch provided by juergen
   http://svn.apache.org/viewvc?rev=675650&view=rev
-  +1: markt
+  +1: markt, remm
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=42565
@@ -73,7 +73,7 @@
   Patch provided by Lucas Galfaso
   This is just the jjt patch. There will also be changes to generated code
   http://svn.apache.org/viewvc?rev=675726&view=rev
-  +1: markt
+  +1: markt, remm
   -1: 
 
 * Fix https://issues.apache.org/bugzilla/show_bug.cgi?id=42722
@@ -81,9 +81,11 @@
   Patch provided by Nils Hammar
   http://svn.apache.org/viewvc?rev=675751&view=rev
   +1: markt
-  -1: 
+  -1: remm: I look at the code, and implicitly it is not the right way to 
close a stream; 
+            this should be in the finally with the usual try { if(!null) } 
catch (IOE)
 
 * Fix startup failure when security is enabled
   http://svn.apache.org/viewvc?rev=675796&view=rev
   +1: markt
   -1: 
+  -0: remm: Does StandardWrapper need to be preloaded now ?



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to