summaryrefslogtreecommitdiff
path: root/diffcore-delta.c
diff options
context:
space:
mode:
authorLibravatar Jeff King <peff@peff.net>2009-12-08 02:49:35 -0500
committerLibravatar Junio C Hamano <gitster@pobox.com>2009-12-07 23:52:46 -0800
commit8947fdd598848b8ceb2cea9ff42e906c946eae5c (patch)
tree9346df8691283dbdb31c2695883851de7848b5e0 /diffcore-delta.c
parentadd-interactive: handle deletion of empty files (diff)
downloadtgif-8947fdd598848b8ceb2cea9ff42e906c946eae5c.tar.xz
add-interactive: fix deletion of non-empty files
Commit 24ab81a fixed the deletion of empty files, but broke deletion of non-empty files. The approach it took was to factor out the "deleted" line from the patch header into its own hunk, the same way we do for mode changes. However, unlike mode changes, we only showed the special "delete this file" hunk if there were no other hunks. Otherwise, the user would annoyingly be presented with _two_ hunks: one for deleting the file and one for deleting the content. This meant that in the non-empty case, we forgot about the deleted line entirely, and we submitted a bogus patch to git-apply (with "/dev/null" as the destination file, but not marked as a deletion). Instead, this patch combines the file deletion hunk and the content deletion hunk (if there is one) into a single deletion hunk which is either staged or not. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'diffcore-delta.c')
0 files changed, 0 insertions, 0 deletions