[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: master c430f7e breaks diff-mode fontification
From: |
Mark Oteiza |
Subject: |
Re: master c430f7e breaks diff-mode fontification |
Date: |
Mon, 27 Jun 2016 16:58:28 -0400 |
User-agent: |
Mutt/1.6.1+53 (91af19866bbd) (2016-04-27) |
On 27/06/16 at 05:01pm, Stephen Berman wrote:
> On Mon, 27 Jun 2016 11:22:50 +0200 Stephen Berman <address@hidden> wrote:
>
> > This change breaks diff-mode fontification:
> >
> >> branch: master
> >> commit c430f7e23fc2c22f251ace4254e37dea1452dfc3
> >> Author: Mark Oteiza <address@hidden>
> >> Commit: Mark Oteiza <address@hidden>
> >>
> >> Remove some face aliases obsoleted in 22.1
> >
> > To reproduce (after git pull and make bootstrap):
> >
> > 0. emacs -Q
> > 1. C-x v L RET <path to emacs master git root> RET
> > 2. In the *vc-change-log* buffer type `d' on one of the entries.
> > => The diff displayed in the *vc-diff* buffer lack the expected faces on
> > the removed and added lines, and the *Messages* buffer contains these
> > messages:
> > Error during redisplay: (jit-lock-function 1) signaled (void-variable
> > diff-hunk-header)
> > Error during redisplay: (jit-lock-function 501) signaled (void-variable
> > diff-removed)
> > Error during redisplay: (jit-lock-function 1001) signaled (void-variable
> > diff-hunk-header)
> > Error during redisplay: (jit-lock-function 1501) signaled (void-variable
> > diff-context)
> >
> > If prior to step 1 above I evaluate the file diff-mode.el from emacs-25
> > (i.e. the version before the above commit), then after 2 the *vc-diff*
> > buffer is fontified as expected and there no such error messages in
> > *Messages*.
>
> The following patch fixes the breakage. (I haven't yet gone through the
> other files changed by commit c430f7e to see if there are similar
> problems.)
Thanks, applied. It's clear to me now why this broke, so I'll have
a closer look as well.
> <snip>