Author: kkolinko
Date: Fri Aug 14 09:38:01 2009
New Revision: 804140

URL: http://svn.apache.org/viewvc?rev=804140&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=804140&r1=804139&r2=804140&view=diff
==============================================================================
--- tomcat/tc6.0.x/trunk/STATUS.txt (original)
+++ tomcat/tc6.0.x/trunk/STATUS.txt Fri Aug 14 09:38:01 2009
@@ -228,8 +228,13 @@
 * Port Active Directory improvements to JNDIREalm from trunk
   Patch testing successfully by willing volunteer on the users list
   http://people.apache.org/~markt/patches/2009-08-06-ADforJNDIRealm.patch
-  +1: markt
+  +1: markt, kkolinko
   -1: 
+  kkolinko: (
+     There are several (two) places with a loop printing containerLog.debug(
+     "Found role: " + it.next()); It would be better to prepare the whole 
string
+     of roles and print it at once.
+  )
 
 * Port TLD processing improvements from trunk
   There have been quite a few changes to TLD processing and they are tightly
@@ -300,7 +305,7 @@
 
 * Make context deployment error message more meaningful
   http://svn.apache.org/viewvc?rev=802940&view=rev
-  +1: markt
+  +1: markt, kkolinko
   -1: 
 
 * Additional patch required for 



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

Reply via email to