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
