Anton Lyxell

See e.g. https://sonarqube.ow2.org/project/issues?id=fr.inria.gforge.spoon%3Aspoon-core&open=AWtJlSdobu-R5OS58r2n&resolved=false&rules=java%3AS3824

Currently some nodes in switch expressions are not included in the ast.

Patches like these are currently unmergeable because they introduce too long lines: ```diff @@ -168,11 +168,7 @@ public class RedissonSessionStore implements SessionStore, Patt @Override public CompletableFuture deleteSession(String id) { return...

`--verbose` should print performance information. Look a bit extra on these files after this is done: ``` spoon/src/test/resources/toStringBugTest/TestClass2.java spoon/src/test/resources/noclasspath/FileIO.java ```

This option should show the steps taken to get to the final diff.

The --accept flag should be changed to work like this: ``` logifix --accept rule1 --accept rule2 ... logifix -a rule1 -a rule2 ``` ``` logifix --accept-all logifix -A ```

Possible future work: Learn the code style from the project under analysis, e.g. how the project handles too long lines, how lambdas are broken up over several lines etc, if...

Improve highlighting in diffs

2

It would be nice with better diff highlighting, as seen in e.g. comby https://www.youtube.com/watch?v=JMZLBB_BFNg (03:20) This could probably be achieved with tokenization and a check between upper and lower lines,...

Good example of where it is needed: https://rules.sonarsource.com/java/type/Bug/RSPEC-1317