Re: [aspectj-users] WeavingURLClassLoader - NPE calling addURL(..) with

[Andy Clement <andrew.clement@xxxxxxxxx>]

Hi, yes I would just grab that class file from the latest dev build.
We are on the verge of 1.6.3 and 1.6.4 appearing in Maven - but those
won't include this change of course.

Feel free to give us any feedback on what you are doing with extending
WeavingURLClassLoader as I know the documentation around doing that is
a bit sketchy.

cheers,
Andy.

2009/4/28 STEVE reds <stevebread@xxxxxxxxx>:
>
> Thanks Andy, that will work just fine. Should I pick up the class file from the next development build? I am currently on 1.6.2 since that is the latest version on the maven repository.
>
> Thanks,
> Steve
>
>
> --- On Mon, 4/27/09, Andy Clement <andrew.clement@xxxxxxxxx> wrote:
>
>> From: Andy Clement <andrew.clement@xxxxxxxxx>
>> Subject: Re: [aspectj-users] WeavingURLClassLoader - NPE calling addURL(..)  without a prior call to define(..)
>> To: aspectj-users@xxxxxxxxxxx
>> Date: Monday, April 27, 2009, 3:38 PM
>> sounds like a bug to me. I just added a guard to addURL() to
>> initialize it if it isn't already setup - does that
>> work for you?
>>
>> Andy.
>>
>> 2009/4/27 STEVE reds <stevebread@xxxxxxxxx>:
>> >
>> > I have a custom ClassLoader that extends
>> WeavingURLClassLoader. In order to initialize the loader
>> from aop.xml, a ClassLoaderWeavingAdaptor reference is
>> needed which is initialized by WeavingURLClassLoader on the
>> first call to defineClass. (To have WeavingURLClassLoader
>> use ClassLoaderWeavingAdaptor instead of WeavingAdaptor, no
>> aspectURLs are passed to the constructor.)
>> >
>> > Since addURL also references the
>> ClassLoaderWeavingAdaptor instance but doesn't check for
>> null, an NPE is generated if addURL is called before
>> defining any class.
>> >
>> > Is this a bug?
>> >
>> >
>> >
>> >
>> > _______________________________________________
>> > aspectj-users mailing list
>> > aspectj-users@xxxxxxxxxxx
>> > https://dev.eclipse.org/mailman/listinfo/aspectj-users
>> >
>> _______________________________________________
>> aspectj-users mailing list
>> aspectj-users@xxxxxxxxxxx
>> https://dev.eclipse.org/mailman/listinfo/aspectj-users
>
>
>
> _______________________________________________
> aspectj-users mailing list
> aspectj-users@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/aspectj-users
>