Bug 309402 - Changes to Main.java
Summary: Changes to Main.java
Status: VERIFIED FIXED
Alias: None
Product: AspectJ
Classification: Tools
Component: Compiler (show other bugs)
Version: unspecified   Edit
Hardware: Macintosh Mac OS X - Carbon (unsup.)
: P3 enhancement (vote)
Target Milestone: 1.6.9M1   Edit
Assignee: aspectj inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-04-15 19:09 EDT by Andrew Eisenberg CLA
Modified: 2010-04-19 15:02 EDT (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Eisenberg CLA 2010-04-15 19:09:23 EDT
In order to get the AJDT ant integration working, we need a small change to Main.java.

We need to be able to pass in a custom org.aspectj.bridge.ICommand object.  Currently, the ICommand object is created via reflection.

Here is my suggestion:

1. augment the ICommand local variable to being a field
2. add a setter for this field
3. change this:
ICommand command = ReflectionFactory.makeCommand(commandName, holder);
to this:
if (command != null)
  command = ReflectionFactory.makeCommand(commandName, holder);
Comment 1 Andrew Clement CLA 2010-04-16 11:18:31 EDT
done
Comment 2 Andrew Eisenberg CLA 2010-04-19 15:02:39 EDT
Yep.  These changes are working for me as expected.  Thanks.