search

vesperin / vesper

source code curation library for Java 1.6 and above
1 stars 0 forks source link

RemoveUnusedMethods is broken #6

Closed hsanchez closed 10 years ago

hsanchez commented 10 years ago

if multiples methods are marked as unused, this class will remove only one from the list of found unused methods. THe ideal scenario is not create a ASTRewrite for the compilation unit and share it among ASTNodes removal. This way there will be ONLY one delta that can be applied by the CommitRequest.

hsanchez commented 10 years ago

is the ideal scenario forcing the user to delete one at a time? hmmm, if yes, then the current implementation works as expected. If it is not, which I favor, then it should be fixed.

hsanchez commented 10 years ago

closing because Pull Request #9