From: Bill Barker [mailto:[EMAIL PROTECTED] 
> From: <[EMAIL PROTECTED]>
> 
> ===================================================================
> >   RCS file:
> /home/cvs/jakarta-tomcat-catalina/catalina/src/share/org/apach
e/catalina/loa
> der/WebappClassLoader.java,v
> >   retrieving revision 1.36
> >   retrieving revision 1.37
> >   diff -u -r1.36 -r1.37
> >   --- WebappClassLoader.java 25 Jun 2004 23:56:25 -0000 1.36
> >   +++ WebappClassLoader.java 3 Jul 2004 18:50:10 -0000 1.37
> >   @@ -1886,8 +1886,12 @@
> >         */
> >        protected boolean isPackageSealed(String name, 
> Manifest man) {
> >
> >   -        String path = name + "/";
> >   -        Attributes attr = man.getAttributes(path);
> >   +        StringBuffer buf = new StringBuffer(name);
> >   +        for (int i=0;i<buf.length();i++) {
> >   +            if (buf.charAt(i)=='.') buf.setCharAt(i,'/');
> >   +        }
> >   +        buf.append('/');
> >   +        Attributes attr = man.getAttributes(buf.toString());
> >            String sealed = null;
> >            if (attr != null) {
> >                sealed = attr.getValue(Name.SEALED);
> 
> It's not a big deal, but wouldn't it be cleaner to do:
>     String path = name.replace('.', '/') + "/";

Fair point. I'll make the change.



---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to