[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH] Showing the relevant part of a diff
From: |
Herring, Davis |
Subject: |
[PATCH] Showing the relevant part of a diff |
Date: |
Fri, 19 Feb 2016 05:43:23 +0000 |
When looking at changes made to a file, the ones most likely to be interesting
are those affecting the portion of the file in view. The second attached patch
causes `vc-diff' to place point at the place in the diff that corresponds to
point in the file being compared (only if the current source is the endpoint of
the diff, of course).
The first patch (starting from a recent master) is preparatory and provides
(and uses) a defconst for the main work.
To resolve:
1. Would this need a NEWS entry?
2. Does it need to be customizable? (M-< is all it takes to "recover the old
behavior"...)
3. Is it worth supporting the (very rare) "normal" diff format?
4. Should M-x diff (or any other diff commands) behave similarly?
5. Do context/unified headers lack the (correct) length of the hunk often
enough to bother with "donttrustheader"? (This is the two FIXMEs in the patch.)
Davis
0001-Clean-up-context.patch
Description: 0001-Clean-up-context.patch
0002-Set-vc-diff-point.patch
Description: 0002-Set-vc-diff-point.patch
- [PATCH] Showing the relevant part of a diff,
Herring, Davis <=