geronimo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Vamsavardhana Reddy" <c1vams...@gmail.com>
Subject Re: svn commit: r487020 - /geronimo/server/branches/2.0-M1/modules/geronimo-jetty6-builder/src/main/java/org/apache/geronimo/jetty6/deployment/JettyModuleBuilder.java
Date Thu, 14 Dec 2006 07:59:32 GMT
Fixed in r487099.


On 12/14/06, Vamsavardhana Reddy <c1vamsi1c@gmail.com> wrote:
>
> On 12/14/06, hogstrom@apache.org <hogstrom@apache.org> wrote:
> >
> > Author: hogstrom
> > Date: Wed Dec 13 22:54:54 2006
> > New Revision: 487020
> >
> > URL: http://svn.apache.org/viewvc?view=rev&rev=487020
> > Log:
> > Applying r486567 from trunk ( 2.0-SNAPSHOT)
> >
> > Modified:
> >
> >     geronimo/server/branches/2.0-M1/modules/geronimo-jetty6-builder/src/main/java/org/apache/geronimo/jetty6/deployment/JettyModuleBuilder.java
> >
> > Modified:
> > geronimo/server/branches/2.0-M1/modules/geronimo-jetty6-builder/src/main/java/org/apache/geronimo/jetty6/deployment/JettyModuleBuilder.java
> >
> > URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0-M1/modules/geronimo-jetty6-builder/src/main/java/org/apache/geronimo/jetty6/deployment/JettyModuleBuilder.java?view=diff&rev=487020&r1=487019&r2=487020
> >
> >
> > ==============================================================================
> > ---
> > geronimo/server/branches/2.0-M1/modules/geronimo-jetty6-builder/src/main/java/org/apache/geronimo/jetty6/deployment/JettyModuleBuilder.java
> > (original)
> > +++
> > geronimo/server/branches/2.0-M1/modules/geronimo-jetty6-builder/src/main/java/org/apache/geronimo/jetty6/deployment/JettyModuleBuilder.java
> > Wed Dec 13 22:54:54 2006
> > @@ -700,7 +700,7 @@
> >              }
> >              UrlPatternType[] urlPatterns =
> > servletMappingType.getUrlPatternArray();
> >              for (int j = 0; j < urlPatterns.length; j++) {
> > -                String urlPattern = urlPatterns[j].getStringValue();
> > +                String urlPattern =
> > urlPatterns[j].getStringValue()i.trim();
>
>
> Won't the above line result in compilation error?
>
>                  if (!knownServletMappings.contains(urlPattern)) {
> >                      knownServletMappings.add(urlPattern);
> >                      checkString(urlPattern);
> >
> >
> >
>

Mime
View raw message