Description Andrew Eisenberg 2009-05-21 16:24:17 EDT

The major work for the push in refactoring is complete, but there are still some smaller issues that I should address.

1. visibility. If a pushed-in ITD references some private state of its original aspect, this will show up as an error after the refactoring is complete. It will be up to the user to fix the visibility.

2. declare parents. pushing in declare parents, I use the same infrastructure as that from AspectsConvertingParser to rewrite a class?s declaration line. This means that pushing in a single declare parents into a type will mean that all parents get pushed in. I think this is a rare enough case that we will only fix this if there is a need.

3. fully qualified names. declare parents and annotations will always insert its type using a fully qualified name, rather than using an import statement

4. would be nice to be able to remove specific ITDs from the refactoring by unchecking them in the refactoring wizard.