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
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
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