Daniil Penkin

Results 13 issues of Daniil Penkin

The first symbol of _each_ line in the hunks is a line type symbol, and for a neutral line it's a space. At the moment it is not stripped out,...

`git.diff` test resource is located under `org/wickedsource/diffparser/unified/`, whereas the `GitDiffTest` class which uses it has been moved at b6df332 to `io.reflectoring.diffparser.unified`, so it is now failing due to resource load...

This removes an extra leading character in a safe manner.

Sorry for a bit of a mess, there seems no way to make a cross-repo stacked pull request on GitHub, so this PR also contains a commit form my [previous...

TestNG version used has JUnit as direct dependencies, and test classes use JUnit's assertions. This is quite fragile and prevents migration to a newer TestNG library.

Both Travis and Gradle builds are broken because of invalid characters in the javadoc: ``` /home/travis/build/thombergs/diffparser/src/main/java/io/reflectoring/diffparser/unified/ParserState.java:26: error: malformed HTML * @author Tom Hombergs ^ /home/travis/build/thombergs/diffparser/src/main/java/io/reflectoring/diffparser/unified/ParserState.java:26: error: bad use of '>'...

The `Diff` class is mutable and hence not thread safe. There's nothing in the way to make it immutable since its instances are essentialy data objects.

The diff tools use a special line to report about absence of the trailing newline character aka [incomplete line](https://www.gnu.org/software/diffutils/manual/html_node/Incomplete-Lines.html) (some report when it differs between the compared files, others report...

The callback passed into `response()` method is called twice in case an exception is raised in the body of that callback. ## Description If the asynchronous callback passed into `response()`...