[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#33567: Syntactic fontification of diff hunks
From: |
Juri Linkov |
Subject: |
bug#33567: Syntactic fontification of diff hunks |
Date: |
Thu, 06 Dec 2018 01:35:17 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (x86_64-pc-linux-gnu) |
>> When this special variable is set by a diff command, it's safe to assume
>> that the files referenced from the diff buffer really exist, so it's
>> safe to try reading them. I don't want a patch in a mail attachment
>> to try reading files mentioned in the patch attachment.
>
> I think you lean heavily towards the use case where the diffs were
> produced by some Emacs command. Whereas what I have in mind is the
> use case where the diffs came from some other source, like manual
> invocation of shell commands. I'm saying that in the absence of
> diff-default-directory, using default-directory and 'cd' will be much
> easier for users.
Users still can use all diff-mode commands to visit source files from them.
Only language syntax fontification in diffs will fall back to hunk-only
when a diff buffer is not created by a diff command. This is to avoid all
possible dangers of automatically visiting files in arbitrary diff buffers.
> The use case I have in mind is that some time passes between the
> generation of the diffs and the time the diffs are visited and
> fontified. During that time, the working revision could have been
> changed. Isn't that what this issue is about? If so, we should
> explain why in that case, rare as it could be, the fontification might
> be wrong.
Fontification can't be wrong because code verifies if hunks from diff
still exist in the updated file, and doesn't highlight changed hunks.
- bug#33567: Syntactic fontification of diff hunks, (continued)
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/10
- bug#33567: Syntactic fontification of diff hunks, Eli Zaretskii, 2018/12/11
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/11
- bug#33567: Syntactic fontification of diff hunks, Eli Zaretskii, 2018/12/12
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/03
- bug#33567: Syntactic fontification of diff hunks, Eli Zaretskii, 2018/12/04
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/04
- bug#33567: Syntactic fontification of diff hunks, Eli Zaretskii, 2018/12/05
- bug#33567: Syntactic fontification of diff hunks,
Juri Linkov <=
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/12
- bug#33567: Syntactic fontification of diff hunks, Eli Zaretskii, 2018/12/14
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/16
- bug#33567: Syntactic fontification of diff hunks, Eli Zaretskii, 2018/12/17
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/17
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/17
- bug#33567: Syntactic fontification of diff hunks, Dmitry Gutov, 2018/12/18
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/18
- bug#33567: Syntactic fontification of diff hunks, Dmitry Gutov, 2018/12/18
- bug#33567: Syntactic fontification of diff hunks, Juri Linkov, 2018/12/18