Re: :extend t inheritance

[Ingo Lohmar]

On Thu, Oct 24 2019 16:46 (+0300), Eli Zaretskii wrote:

> I don't think I understand what you are saying or asking.  The text
> that you cite doesn't mention inheritance at all.  Could you clarify
> the issue?

Sorry, my mistake: I forgot a) that I had known about face merging
before, and b) the behavior described in the text seemed to fit what I
was seeing with inheritance of the :extend attribute.

On Thu, Oct 24 2019 17:38 (+0200), Kévin Le Gouguec wrote:
> From emacs -Q:
>
> Add and remove a bunch of lines in a version-controlled file, and open a
> *vc-diff* buffer with C-x v =.  Both diff-added and diff-removed faces
> extend beyond EOL since by default their :extend attribute is set to t.
>
> Then evaluate:
>
> (set-face-extend 'diff-added 'unspecified)
> (set-face-extend 'diff-removed 'unspecified)
> (set-face-extend 'diff-changed t)
>
> Intended effect: none, since diff-added and diff-removed inherit
> diff-changed.
>
> Actual effect: diff-added and diff-removed no longer extend beyond EOL.

Thanks a lot for this recipe!