]> source.dussan.org Git - gitea.git/commit
Fix broken spans in diffs (#14678)
authorzeripath <art27@cantab.net>
Sun, 14 Feb 2021 14:51:00 +0000 (14:51 +0000)
committerGitHub <noreply@github.com>
Sun, 14 Feb 2021 14:51:00 +0000 (15:51 +0100)
commitbeb2058186e81a49dbe7932123bf63bbe1f32d35
treeca9a7a4d1640c56e0de58aa0d9644cfa1f24659a
parentf3847c9d8203021732c331431e0755f98ca260de
Fix broken spans in diffs (#14678)

Gitea runs diff on highlighted code fragment for each line in order to provide
code highlight diffs. Unfortunately this diff algorithm is not aware that span tags
and entities are atomic and cannot be split.

The current fixup code makes some attempt to fix these broken tags however, it cannot
handle situations where a tag is split over multiple blocks.

This PR provides a more algorithmic fixup mechanism whereby spans and entities are
completely coalesced into their respective blocks.

This may result in a incompletely reduced diff but - it will definitely prevent the
broken entities and spans that are currently possible.

As a result of this fixup several inconsistencies were discovered in our testcases
and these were also fixed.

Fix #14231

Signed-off-by: Andrew Thornton <art27@cantab.net>
services/gitdiff/gitdiff.go
services/gitdiff/gitdiff_test.go