Bug 216311 - ITD fields on interfaces end up transient with annotation style
Summary: ITD fields on interfaces end up transient with annotation style
Status: RESOLVED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: Compiler (show other bugs)
Version: 1.5.4RC1   Edit
Hardware: PC All
: P2 critical with 1 vote (vote)
Target Milestone: 1.6.0 M2   Edit
Assignee: aspectj inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-01-23 12:47 EST by Guillaume Lasnier CLA
Modified: 2008-02-20 15:01 EST (History)
1 user (show)

See Also:

Attachments
The IPersistable interface to be extended by the aspect (142 bytes, text/java-source)
2008-01-23 12:47 EST, Guillaume Lasnier CLA 		no flags Details
The ITD aspect using Java @Annotations (1019 bytes, text/java-source)
2008-01-23 12:48 EST, Guillaume Lasnier CLA 		no flags Details
The serialization test case (JUnit 4) (1.06 KB, text/java-source)
2008-01-23 12:49 EST, Guillaume Lasnier CLA 		no flags Details
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Guillaume Lasnier CLA 2008-01-23 12:47:49 EST 
Created attachment 87678 [details]
The IPersistable interface to be extended by the aspect

+++ This bug was initially created as a clone of Bug #168063 +++

The same problem occurs when using @Annotations style.
Comment 1 Guillaume Lasnier CLA 2008-01-23 12:48:34 EST 
Created attachment 87679 [details]
The ITD aspect using Java @Annotations
Comment 2 Guillaume Lasnier CLA 2008-01-23 12:49:06 EST 
Created attachment 87680 [details]
The serialization test case (JUnit 4)
Comment 3 Andrew Clement CLA 2008-02-20 15:01:22 EST 
fix committed.  Changed the instance field for the aspect instance that is added to the target type to include an extra component in its name, instead of just 'ajc$' as a prefix, it is now 'ajc$instance$' which enables the logic that checks for what should be made transient to identify it and *not* make it transient.