Proper handling of newlines when parsing comments

[koppor]

I am investigating another "interesting" PR (JabRef/jabref-koppor#687)

I noticed that comments are now handled. Thank you for that!

It would be great if RefactoringMinor could handle newlines. Currently, if "only" line breaks change, it is determined as deleted/new - not as moved:

[tsantalis]

Move code from main to initLogging

[tsantalis]

@koppor

It's really cool the tool finds that routeLoggingToSlf4J and addLogToDisk have been merged to initLogging
On top of that initializeLogger has been inlined to initLogging.

This is a challenging case, but our tool did pretty well !!

[tsantalis]

Diff after supporting move code from main to initLogging

[koppor]

Thank you for continuosly improving the tool! - (I still miss the connecting lines (#703 (comment)) to see which block went where). I am still used to the IntelliJ "Resolve Conflicts" dialog ^^)

[tsantalis]

@koppor @pouryafard75
The javadoc is matched as follows:

[tsantalis]

@pouryafard75

4 of our tests failed after this update.
I checked them and all of them have reformatted Javadoc.
So, the change is positive. I will check if the Javadoc mappings can be further improved.

org.opentest4j.AssertionFailedError: Failed for belaban/JGroups@f153375 , srcFileName: src/org/jgroups/auth/FixedMembershipToken.java ==> expected: <112842> but was: <113219>

org.opentest4j.AssertionFailedError: Failed for spring-projects/spring-security@fcc9a34 , srcFileName: web/src/main/java/org/springframework/security/web/context/SaveContextOnUpdateOrErrorResponseWrapper.java ==> expected: <442150> but was: <443825>

org.opentest4j.AssertionFailedError: Failed for addthis/hydra@7fea4c9 , srcFileName: hydra-main/src/main/java/com/addthis/hydra/job/spawn/SpawnBalancer.java ==> expected: <2844989> but was: <2846125>

org.opentest4j.AssertionFailedError: Failed for AsyncHttpClient/async-http-client@f01d861 , srcFileName: api/src/test/java/org/asynchttpclient/oauth/OAuthSignatureCalculatorTest.java ==> expected: <219056> but was: <221438>

org.opentest4j.AssertionFailedError: Failed for nutzam/nutz@de7efe4 , srcFileName: src/org/nutz/dao/util/Daos.java ==> expected: <1103701> but was: <1103968>

[tsantalis]

@koppor @pouryafard75

This issue has been fixed to perfection!
Thanks Pouria for your nice implementation.