Re: svn commit: r1028521 - in /tomcat/trunk: java/org/apache/catalina/ java/org/apache/catalina/connector/ java/org/apache/catalina/core/ java/org/apache/tomcat/util/http/mapper/ test/org/apache/tomca

2010-10-29 Thread Tim Funk 
Should we add trim()? (in case the user has a new line or spaces after 
the ,)


this.resourceOnlyServlets.add(servletName.trim());


-Tim

Modified: tomcat/trunk/java/org/apache/catalina/core/StandardContext.java
URL: 
http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardContext.java?rev=1028521&r1=1028520&r2=1028521&view=diff

==
--- tomcat/trunk/java/org/apache/catalina/core/StandardContext.java 
(original)
+++ tomcat/trunk/java/org/apache/catalina/core/StandardContext.java Thu 
Oct 28 22:29:25 2010


On 10/28/2010 6:29 PM, ma...@apache.org wrote:

+@Override
+public void setResourceOnlyServlets(String resourceOnlyServlets) {
+this.resourceOnlyServlets.clear();
+if (resourceOnlyServlets == null ||
+resourceOnlyServlets.length() == 0) {
+return;
+}
+String[] servletNames = resourceOnlyServlets.split(",");
+for (String servletName : servletNames) {
+this.resourceOnlyServlets.add(servletName);
+}
+}


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

Re: svn commit: r1028521 - in /tomcat/trunk: java/org/apache/catalina/ java/org/apache/catalina/connector/ java/org/apache/catalina/core/ java/org/apache/tomcat/util/http/mapper/ test/org/apache/tomca

2010-10-29 Thread Mark Thomas 
On 29/10/2010 13:51, Tim Funk wrote:
> Should we add trim()? (in case the user has a new line or spaces after
> the ,)
> 
> this.resourceOnlyServlets.add(servletName.trim());

Can't hurt. Go for it.

Mark

> 
> 
> -Tim
> 
> Modified: tomcat/trunk/java/org/apache/catalina/core/StandardContext.java
> URL:
> http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/catalina/core/StandardContext.java?rev=1028521&r1=1028520&r2=1028521&view=diff
> 
> ==
> 
> --- tomcat/trunk/java/org/apache/catalina/core/StandardContext.java
> (original)
> +++ tomcat/trunk/java/org/apache/catalina/core/StandardContext.java Thu
> Oct 28 22:29:25 2010
> 
> On 10/28/2010 6:29 PM, ma...@apache.org wrote:
>> +@Override
>> +public void setResourceOnlyServlets(String resourceOnlyServlets) {
>> +this.resourceOnlyServlets.clear();
>> +if (resourceOnlyServlets == null ||
>> +resourceOnlyServlets.length() == 0) {
>> +return;
>> +}
>> +String[] servletNames = resourceOnlyServlets.split(",");
>> +for (String servletName : servletNames) {
>> +this.resourceOnlyServlets.add(servletName);
>> +}
>> +}
> 
> -
> To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org
> For additional commands, e-mail: dev-h...@tomcat.apache.org
> 


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