-
Notifications
You must be signed in to change notification settings - Fork 82
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Review/port performance enhancements proposed in bugzilla issue #9
Comments
Any update? This seems like a potentially big improvement |
Unfortunately haven't had time to do it, but if someone wants to turn it into a PR with maybe a system property to control the behaviour, that would be a likely way for it to go in with limited testing (so by default you get todays behaviour, but switch on a property and this feature could be tested out). |
Because the commits were directly on master and do not come from PR #37, let me mention for reference that @aclement added these changes: 1b3cead...3227aad which are probably going to superseed the PR and eventuall close this issue. Andy was also so kind to add basic tests. 😊 |
To do: Mention the new options in the 1.9.8 release notes and wherever else it might make sense to integrate them into existing documentation. |
I think yes to release notes, obviously, but also it needs its own post on the mailing list to raise awareness for anyone that wants to try this out and see if it helps them. I'm not sure where else we could publicize it - my AspectJ blog is rather long in the tooth now. |
See https://bugs.eclipse.org/bugs/show_bug.cgi?id=565450 which includes the patch.
The text was updated successfully, but these errors were encountered: