[prev in list] [next in list] [prev in thread] [next in thread]
List: git
Subject: [PATCH v3 2/2] git-gui: support for diff3 conflict style
From: Bert Wesarg <bert.wesarg () googlemail ! com>
Date: 2019-09-30 19:54:53
Message-ID: 14754a59ecf15194dccc659072e2bc180280d097.1569845908.git.bert.wesarg () googlemail ! com
[Download RAW message or body]
This adds highlight support for the diff3 conflict style.
The common pre-image will be reversed to --, because it has been removed
and replaced with ours or theirs side respectively.
Signed-off-by: Bert Wesarg <bert.wesarg@googlemail.com>
---
git-gui.sh | 3 +++
lib/diff.tcl | 17 ++++++++++++++++-
2 files changed, 19 insertions(+), 1 deletion(-)
---
v3: Fixed a syntax error
diff --git a/git-gui.sh b/git-gui.sh
index fd476b6..6d80f82 100755
--- a/git-gui.sh
+++ b/git-gui.sh
@@ -3581,6 +3581,9 @@ $ui_diff tag conf d_s- \
$ui_diff tag conf d< \
-foreground orange \
-font font_diffbold
+$ui_diff tag conf d| \
+ -foreground orange \
+ -font font_diffbold
$ui_diff tag conf d= \
-foreground orange \
-font font_diffbold
diff --git a/lib/diff.tcl b/lib/diff.tcl
index 0fd4600..dacdda2 100644
--- a/lib/diff.tcl
+++ b/lib/diff.tcl
@@ -347,6 +347,10 @@ proc start_show_diff {cont_info {add_opts {}}} {
}
set ::current_diff_inheader 1
+ # detect pre-image lines of the diff3 conflict-style, they are just '++'
+ # lines which is not bijective, thus we need to maintain a state across
+ # lines
+ set ::conflict_in_pre_image 0
fconfigure $fd \
-blocking 0 \
-encoding [get_path_encoding $path] \
@@ -449,11 +453,22 @@ proc read_diff {fd conflict_size cont_info} {
{--} {set tags d_--}
{++} {
set regexp [string map [list %conflict_size $conflict_size]\
- {^\+\+([<>=]){%conflict_size}(?: |$)}]
+ {^\+\+([<>=|]){%conflict_size}(?: |$)}]
if {[regexp $regexp $line _g op]} {
set is_conflict_diff 1
set line [string replace $line 0 1 { }]
set tags d$op
+ # the ||| conflict-marker marks the start of the pre-image,
+ # all those lines are also prefixed with '++', thus we need
+ # to maintain this state
+ set ::conflict_in_pre_image [expr {$op eq {|}}]
+ } elseif {$::conflict_in_pre_image} {
+ # this is a pre-image line, it is the one which both sides
+ # are based on. As it has also the '++' line start, it is
+ # normally shown as 'added', invert this to '--' to make
+ # it a 'removed' line
+ set line [string replace $line 0 1 {--}]
+ set tags d_--
} else {
set tags d_++
}
--
2.23.0.11.g242cf7f110
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic