Removed it because Jetty was expecting a non null. I was going to have it return a empty array, but did not understand why we would want to mask the urls in a url cl.
--jason On Sun, 2002-02-17 at 13:12, Scott M Stark wrote: > > This change to UnifiedClassLoader: > "removed invalid getURLs() method. Not sure why we want to return null > for a URLClassLoader. Comments suggested it was to avoid some RMI bloat, > but code that expects a URLClassLoader will expect that this baby will > return something valid. It certainly won't expect a null. If this > behavior of masking the url list is trully desired, then return new > URL[0], > but I suggest that is also invalid, since a URLClassLoader with no URLS > is > a wasted object... what does one load from it... IMHO (ok not so humble) > it is best not to obfuscate here." > > breaks dynamic class loading and does cause bloat of serialized RMI > objects as well as MarshalledObjects. Do not rip out code you do > not like unless you really understand why it was there. > > xxxxxxxxxxxxxxxxxxxxxxxx > Scott Stark > Chief Technology Officer > JBoss Group, LLC > xxxxxxxxxxxxxxxxxxxxxxxx > > > _______________________________________________ > Jboss-development mailing list > [EMAIL PROTECTED] > https://lists.sourceforge.net/lists/listinfo/jboss-development _______________________________________________ Jboss-development mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/jboss-development
