Bug 356594 - Enhance "declare @*" to take multiple annotations instead of only one
Summary: Enhance "declare @*" to take multiple annotations instead of only one
Status: NEW
Alias: None
Product: AspectJ
Classification: Tools
Component: Compiler (show other bugs)
Version: unspecified   Edit
Hardware: All All
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: aspectj inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-02 09:31 EDT by Matthew Adams CLA
Modified: 2011-09-02 09:39 EDT (History)
0 users

See Also:

Attachments
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 Matthew Adams CLA 2011-09-02 09:31:19 EDT 
Build Identifier: 1.6.11

Instead of:

public aspect PersistableAspect {

    declare @type:
	(@Persistable *):
	@Entity;
	
    declare @type:
	(@Persistable *):
	@Access(AccessType.FIELD);
}

This would be nice:

public aspect PersistableAspect {

    declare @type:
	(@Persistable *):
	@Entity, @Access(AccessType.FIELD);
}

Removes code duplication.


Reproducible: Always
Comment 1 Matthew Adams CLA 2011-09-02 09:39:06 EDT 
On second thought, scratch comma separation.  Just go with whitespace separation, in the same way that annotations currently do.

public aspect PersistableAspect {

    declare @type:
    (@Persistable *):
    @Entity
    @Access(AccessType.FIELD);
}