[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[List Home]
|
Re: [aspectj-users] Issues with @DeclareParents and @DeclareMixin
|
- From: Andy Clement <andrew.clement@xxxxxxxxx>
- Date: Wed, 17 Jun 2009 11:16:35 -0700
- Delivered-to: aspectj-users@eclipse.org
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:content-type :content-transfer-encoding; bh=vTLDLjQp8KdV2bq1WvBeqhiWwx2dk9Qrqcu8KoeC1XE=; b=b8wQgmWXv6wCQYAuLtzwMJJm+ybUZBQuC5AhLqDMwJabisVYWKm84rs5+Q0jVbbAg9 z9YAnEpIeVZrRKOacends6LO5fYsAJlUTTWHflgDXzrPGPKJc7yriZg8LQqFw17Hi78q Fvo9OX765BlhySho7oiLC4ANCJurFRuhL+brU=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; b=tJrWEiszsxHoLc5qCxqldt6uv/j41sVRIW5MiELMCw61B7nzpPFkB3DWPUj7e96gBc ETnMhEhhWRr6+h2yCtg6B4ItyaKsju3q6FXqoqbVHnXbFSjaLpzyRfhwR2PBcnNT5EQ4 wbeCT+VuBQN2UgTN9eVl57BiczQUO0ne7+tDU=
Two bugs raised against AJ now
@DeclareMixin issues: https://bugs.eclipse.org/bugs/show_bug.cgi?id=280651
@DeclareParents issues: https://bugs.eclipse.org/bugs/show_bug.cgi?id=280658
The former is higher priority because I consider @DeclareParents
deprecated in favour of DeclareMixin.
Andy
2009/6/17 Andy Clement <andrew.clement@xxxxxxxxx>:
>
>
> 2009/6/17 Dunstan, Tom (SACE Board) <TomD@xxxxxxxxxxxxxxxxxxx>
>>
>> Just for fun, I then uninstalled the AJDT and installed the dev version
>> 2.0.0.e34x-20090616-1700 which bundles AspectJ version:
>> 1.6.5.20090612153600. Both aspects apply fine to classes in the same
>> project. When trying to apply them to my client project, the ParentsAspect
>> does the same thing as 1.6.4, but the MixinAspect causes an AspectJ Internal
>> Compiler Error:
>>
>> java.lang.NullPointerException
>> at
>> org.aspectj.asm.internal.ProgramElement.addChild(ProgramElement.java:398)
>> at
>> org.aspectj.weaver.model.AsmRelationshipProvider.addChildNodes(AsmRelationshipProvider.java:611)
>> at
>> org.aspectj.weaver.model.AsmRelationshipProvider.createHierarchy(AsmRelationshipProvider.java:425)
>> at
>> org.aspectj.weaver.model.AsmRelationshipProvider.addRelationship(AsmRelationshipProvider.java:121)
>> at org.aspectj.weaver.bcel.B ... OKEINTERFACE java.lang.Runnable.run ()V
>> RETURN (line 17)
>> end public static void main(String[])
>> end public class ctest.Foo
>
> The @DeclareMixin has had basic testing, but looks like you found a bug with
> it - i'll fix this today (if I can recreate it). I kind of see
> @DeclareMixin as a replacement for @DeclareParents as it offers a nicer more
> flexible syntax - so I'd prefer it that approach was used moving forward.
> Do you have a preference?
>
> Andy
>