GNU bug report logs - #20245
24.5; c++-mode font-lock broken by the last commit to emacs24 branch

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 20245 in the body.
You can then email your comments to 20245 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Oleh Krehel <ohwoeowho <at> gmail.com>
To: bug-gnu-emacs <at> gnu.org
Cc: acm <at> muc.de
Subject: 24.5; c++-mode font-lock broken by the last commit to emacs24 branch
Date: Thu, 02 Apr 2015 08:55:58 +0200

Hello,

The font-locking of at least C++ headers was broken in "d69e9f1".
Here's a screenshot: http://i.imgur.com/eT6X3hP.png. You can see three
artifacts there. They change number and location when I scroll.

It goes back to a good state if I revert the commit and re-compile.
I hope this gets fixed by 24.5 release, even by just reverting
"d69e9f1", since this one is a bigger bug than
http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19669, which "d69e9f1" fixes.

Oleh

Message #8 received at 20245 <at> debbugs.gnu.org (full text, mbox):

From: Alan Mackenzie <acm <at> muc.de>
To: 20245 <at> debbugs.gnu.org
Cc: Oleh Krehel <ohwoeowho <at> gmail.com>
Subject: Re: bug#20245: 24.5;
 c++-mode font-lock broken by the last commit to emacs24 branch
Date: 2 Apr 2015 09:51:35 -0000

Hello, Oleh.

In article <mailman.3171.1427958137.31049.bug-gnu-emacs <at> gnu.org> you wrote:

> The font-locking of at least C++ headers was broken in "d69e9f1".
> Here's a screenshot: http://i.imgur.com/eT6X3hP.png. You can see three
> artifacts there. They change number and location when I scroll.

This was fixed very recently in the master branch.  Would you please try
the same file in master to confirm that the fix is a fix.

> It goes back to a good state if I revert the commit and re-compile.
> I hope this gets fixed by 24.5 release, even by just reverting
> "d69e9f1", since this one is a bigger bug than
> http://debbugs.gnu.org/cgi/bugreport.cgi?bug=19669, which "d69e9f1" fixes.

> Oleh

-- 
Alan Mackenzie (Nuremberg, Germany).

Message #11 received at 20245 <at> debbugs.gnu.org (full text, mbox):

From: Oleh Krehel <ohwoeowho <at> gmail.com>
To: Alan Mackenzie <acm <at> muc.de>
Cc: 20245 <at> debbugs.gnu.org
Subject: Re: bug#20245: 24.5; c++-mode font-lock broken by the last commit to
 emacs24 branch
Date: Thu, 2 Apr 2015 12:19:36 +0200

Hi, Alan.


On Thu, Apr 2, 2015 at 11:51 AM, Alan Mackenzie <acm <at> muc.de> wrote:
> Hello, Oleh.
>
> In article <mailman.3171.1427958137.31049.bug-gnu-emacs <at> gnu.org> you wrote:
>
>> The font-locking of at least C++ headers was broken in "d69e9f1".
>> Here's a screenshot: http://i.imgur.com/eT6X3hP.png. You can see three
>> artifacts there. They change number and location when I scroll.
>
> This was fixed very recently in the master branch.  Would you please try
> the same file in master to confirm that the fix is a fix.

I just tried "23bb07e", the latest commit on master, but this issue isn't fixed.
The include statements look the same as in the screenshot.

Oleh

Message #14 received at 20245 <at> debbugs.gnu.org (full text, mbox):

From: Alan Mackenzie <acm <at> muc.de>
To: Oleh Krehel <ohwoeowho <at> gmail.com>
Cc: 20245 <at> debbugs.gnu.org
Subject: Re: bug#20245: 24.5;
 c++-mode font-lock broken by the last commit to emacs24 branch
Date: 2 Apr 2015 12:48:46 -0000

Hello, Oleh.

In article <mailman.3178.1427970015.31049.bug-gnu-emacs <at> gnu.org> you wrote:
> Hi, Alan.

> On Thu, Apr 2, 2015 at 11:51 AM, Alan Mackenzie <acm <at> muc.de> wrote:
>> In article <mailman.3171.1427958137.31049.bug-gnu-emacs <at> gnu.org> you wrote:

>>> The font-locking of at least C++ headers was broken in "d69e9f1".
>>> Here's a screenshot: http://i.imgur.com/eT6X3hP.png. You can see three
>>> artifacts there. They change number and location when I scroll.

>> This was fixed very recently in the master branch.  Would you please try
>> the same file in master to confirm that the fix is a fix.

> I just tried "23bb07e", the latest commit on master, but this issue isn't fixed.
> The include statements look the same as in the screenshot.

That's a pity.  It looks like there's a separate problem with the "fix"
from February.

Is there any chance you could send me a copy of the file which triggers
the bug, a description of exactly what you do to see it, and (unless you
start from emacs -Q) a copy of your CC Mode configuration (generated by
C-c C-b)?

Thanks!

> Oleh

-- 
Alan Mackenzie (Nuremberg, Germany).

Message #17 received at 20245 <at> debbugs.gnu.org (full text, mbox):

From: Oleh Krehel <ohwoeowho <at> gmail.com>
To: Alan Mackenzie <acm <at> muc.de>
Cc: 20245 <at> debbugs.gnu.org
Subject: Re: bug#20245: 24.5; c++-mode font-lock broken by the last commit to
 emacs24 branch
Date: Thu, 2 Apr 2015 15:10:57 +0200

Hi Alan,

On Thu, Apr 2, 2015 at 2:48 PM, Alan Mackenzie <acm <at> muc.de> wrote:

>> I just tried "23bb07e", the latest commit on master, but this issue isn't fixed.
>> The include statements look the same as in the screenshot.
>
> That's a pity.  It looks like there's a separate problem with the "fix"
> from February.
>
> Is there any chance you could send me a copy of the file which triggers
> the bug, a description of exactly what you do to see it, and (unless you
> start from emacs -Q) a copy of your CC Mode configuration (generated by
> C-c C-b)?

The bug is reproducible with emacs -Q. You can get an example file at:
https://github.com/dune-project/dune-pdelab-howto/blob/master/src/navier-stokes/cgstokes_instat.cc

To trigger the bug, just M->, and then scroll up with the mouse to the
start of the file where the headers are.  Random headers will be
partially highlighted with 'default instead of 'font-lock-string-face.

Oleh

Message #20 received at 20245 <at> debbugs.gnu.org (full text, mbox):

From: Alan Mackenzie <acm <at> muc.de>
To: Oleh Krehel <ohwoeowho <at> gmail.com>
Cc: 20245 <at> debbugs.gnu.org
Subject: Re: bug#20245: 24.5; c++-mode font-lock broken by the last commit to
 emacs24 branch
Date: Thu, 2 Apr 2015 13:32:45 +0000

Hello, Oleh.

On Thu, Apr 02, 2015 at 03:10:57PM +0200, Oleh Krehel wrote:
> The bug is reproducible with emacs -Q. You can get an example file at:
> https://github.com/dune-project/dune-pdelab-howto/blob/master/src/navier-stokes/cgstokes_instat.cc

> To trigger the bug, just M->, and then scroll up with the mouse to the
> start of the file where the headers are.  Random headers will be
> partially highlighted with 'default instead of 'font-lock-string-face.

Thanks.  I've got that file, and I do indeed see the bug in it.

> Oleh

-- 
Alan Mackenzie (Nuremberg, Germany).

Message #25 received at 20245-done <at> debbugs.gnu.org (full text, mbox):

From: Alan Mackenzie <acm <at> muc.de>
To: Oleh Krehel <ohwoeowho <at> gmail.com>
Cc: 20245-done <at> debbugs.gnu.org
Subject: Re: bug#20245: 24.5; c++-mode font-lock broken by the last commit to
 emacs24 branch
Date: Fri, 3 Apr 2015 12:46:53 +0000

Hello, Oleh.

On Thu, Apr 02, 2015 at 12:19:36PM +0200, Oleh Krehel wrote:
> On Thu, Apr 2, 2015 at 11:51 AM, Alan Mackenzie <acm <at> muc.de> wrote:
> > In article <mailman.3171.1427958137.31049.bug-gnu-emacs <at> gnu.org> you wrote:

> >> The font-locking of at least C++ headers was broken in "d69e9f1".
> >> Here's a screenshot: http://i.imgur.com/eT6X3hP.png. You can see three
> >> artifacts there. They change number and location when I scroll.

> > This was fixed very recently in the master branch.  Would you please try
> > the same file in master to confirm that the fix is a fix.

> I just tried "23bb07e", the latest commit on master, but this issue isn't fixed.
> The include statements look the same as in the screenshot.

I've looked at this problem and now understand its cause.  It's too
involved to fix for Emacs 24.5, so we've decided to revert that commit,
as you've suggested.  A proper fix will surely be included in Emacs 25.1
(or 24.6, whichever comes first).

Thanks for taking the trouble to report this bug.  I'm closing it now.

> Oleh

-- 
Alan Mackenzie (Nuremberg, Germany).

Message #28 received at 20245 <at> debbugs.gnu.org (full text, mbox):

From: Alan Mackenzie <acm <at> muc.de>
To: 20245 <at> debbugs.gnu.org, ohwoeowho <at> gmail.com
Subject: Re: bug#20245: 24.5; c++-mode font-lock broken by the last commit to
 emacs24 branch
Date: Mon, 6 Apr 2015 22:40:04 +0000

Hello, Oleh.

On Fri, Apr 03, 2015 at 12:46:53PM +0000, Alan Mackenzie wrote:

> On Thu, Apr 02, 2015 at 12:19:36PM +0200, Oleh Krehel wrote:
> > On Thu, Apr 2, 2015 at 11:51 AM, Alan Mackenzie <acm <at> muc.de> wrote:
> > > In article <mailman.3171.1427958137.31049.bug-gnu-emacs <at> gnu.org> you wrote:

> > I just tried "23bb07e", the latest commit on master, but this issue isn't fixed.
> > The include statements look the same as in the screenshot.

> I've looked at this problem and now understand its cause.  It's too
> involved to fix for Emacs 24.5, so we've decided to revert that commit,
> as you've suggested.  A proper fix will surely be included in Emacs 25.1
> (or 24.6, whichever comes first).

I've now committed a (hopefully) proper fix for this into the master
branch at savannah.  Feel free to download this and check that it does
indeed work properly.  If not ...

> > Oleh

-- 
Alan Mackenzie (Nuremberg, Germany).

Message #31 received at 20245 <at> debbugs.gnu.org (full text, mbox):

From: Oleh Krehel <ohwoeowho <at> gmail.com>
To: Alan Mackenzie <acm <at> muc.de>
Cc: 20245 <at> debbugs.gnu.org
Subject: Re: bug#20245: 24.5;
 c++-mode font-lock broken by the last commit to emacs24 branch
Date: Tue, 07 Apr 2015 14:17:18 +0200

Alan Mackenzie <acm <at> muc.de> writes:

Hi Alan,

> On Fri, Apr 03, 2015 at 12:46:53PM +0000, Alan Mackenzie wrote:
>
> I've now committed a (hopefully) proper fix for this into the master
> branch at savannah.  Feel free to download this and check that it does
> indeed work properly.  If not ...

It works great. Thanks a lot.

regards,
Oleh

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.