unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#40658: Fforward-comment: Incorrect recognition of end of comment in C++
@ 2020-04-16 12:44 Alan Mackenzie
  2020-04-16 13:35 ` Eli Zaretskii
       [not found] ` <handler.40658.B.158704107627532.ack@debbugs.gnu.org>
  0 siblings, 2 replies; 5+ messages in thread
From: Alan Mackenzie @ 2020-04-16 12:44 UTC (permalink / raw)
  To: 40658; +Cc: Stefan Monnier

Hello, Emacs.

C++ comments can be continued onto the next line by an escaped NL.  This
is configured in Emacs with the variable comment-end-can-be-escaped.

However, Fforward_comment (in the backward direction) doesn't take this
into account.  A BOL following an escaped NL cannot be at the end of a
C++ line comment:

    // C++ comment first line \
    Second line of comment
    ^
    |
  point

Point here is in the middle of a comment.

It is therefore wrong in Fforward_comment not to check
comment-end-can-be-escaped when immediately after an Sendcomment.  This
can be fixed thusly:


diff --git a/src/syntax.c b/src/syntax.c
index e24b98da32..ff125b137c 100644
--- a/src/syntax.c
+++ b/src/syntax.c
@@ -2572,8 +2572,9 @@ between them, return t; otherwise return nil.  */)
 	    }
 	  else if (code == Sendcomment)
 	    {
-	      found = back_comment (from, from_byte, stop, comnested, comstyle,
-				    &out_charpos, &out_bytepos);
+              found = (!quoted || !Vcomment_end_can_be_escaped)
+                && back_comment (from, from_byte, stop, comnested, comstyle,
+                                 &out_charpos, &out_bytepos);
 	      if (!found)
 		{
 		  if (c == '\n')


This should clearly go into the master branch.  I think it should also go
into the emacs-27 release branch, since it is relevant for a nasty bug,
bug #40052.  There, scrolling in a C++ include file, which consists of a
#define macro ~2,400 lines long is deadly slow.  With the above fix in
place, scrolling is over 4 times as fast, although still too slow.

-- 
Alan Mackenzie (Nuremberg, Germany).





^ permalink raw reply related	[flat|nested] 5+ messages in thread

* bug#40658: Fforward-comment: Incorrect recognition of end of comment in C++
  2020-04-16 12:44 bug#40658: Fforward-comment: Incorrect recognition of end of comment in C++ Alan Mackenzie
@ 2020-04-16 13:35 ` Eli Zaretskii
  2020-04-16 17:29   ` Alan Mackenzie
       [not found] ` <handler.40658.B.158704107627532.ack@debbugs.gnu.org>
  1 sibling, 1 reply; 5+ messages in thread
From: Eli Zaretskii @ 2020-04-16 13:35 UTC (permalink / raw)
  To: Alan Mackenzie; +Cc: monnier, 40658

> Date: Thu, 16 Apr 2020 12:44:26 +0000
> From: Alan Mackenzie <acm@muc.de>
> Cc: Stefan Monnier <monnier@iro.umontreal.ca>
> 
> This should clearly go into the master branch.  I think it should also go
> into the emacs-27 release branch, since it is relevant for a nasty bug,
> bug #40052.

Sorry, too late for that now.  This code is there for the last 20+
years, we can endure that a bit more, I think.





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#40658: Fforward-comment: Incorrect recognition of end of comment in C++
  2020-04-16 13:35 ` Eli Zaretskii
@ 2020-04-16 17:29   ` Alan Mackenzie
  2020-04-16 18:01     ` Stefan Monnier
  0 siblings, 1 reply; 5+ messages in thread
From: Alan Mackenzie @ 2020-04-16 17:29 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: monnier, 40658

Hello, Eli.

On Thu, Apr 16, 2020 at 16:35:14 +0300, Eli Zaretskii wrote:
> > Date: Thu, 16 Apr 2020 12:44:26 +0000
> > From: Alan Mackenzie <acm@muc.de>
> > Cc: Stefan Monnier <monnier@iro.umontreal.ca>

> > This should clearly go into the master branch.  I think it should also go
> > into the emacs-27 release branch, since it is relevant for a nasty bug,
> > bug #40052.

> Sorry, too late for that now.  This code is there for the last 20+
> years, we can endure that a bit more, I think.

OK, I've committed the patch to master.  The code, in effect, just dates
from 2015, when I added support for escaped EOLs in comments.

I will put some workaround into CC Mode (which I would have to have done
anyway).

-- 
Alan Mackenzie (Nuremberg, Germany).





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#40658: (Fforward-comment: Incorrect recognition of end of comment in C++)
       [not found] ` <handler.40658.B.158704107627532.ack@debbugs.gnu.org>
@ 2020-04-16 17:31   ` Alan Mackenzie
  0 siblings, 0 replies; 5+ messages in thread
From: Alan Mackenzie @ 2020-04-16 17:31 UTC (permalink / raw)
  To: 40658-done

Bug fixed.

-- 
Alan Mackenzie (Nuremberg, Germany).





^ permalink raw reply	[flat|nested] 5+ messages in thread

* bug#40658: Fforward-comment: Incorrect recognition of end of comment in C++
  2020-04-16 17:29   ` Alan Mackenzie
@ 2020-04-16 18:01     ` Stefan Monnier
  0 siblings, 0 replies; 5+ messages in thread
From: Stefan Monnier @ 2020-04-16 18:01 UTC (permalink / raw)
  To: Alan Mackenzie; +Cc: 40658

> I will put some workaround into CC Mode

Even better, thanks.

> (which I would have to have done anyway).

Well, there's no such obligation, really ;-)


        Stefan






^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2020-04-16 18:01 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-04-16 12:44 bug#40658: Fforward-comment: Incorrect recognition of end of comment in C++ Alan Mackenzie
2020-04-16 13:35 ` Eli Zaretskii
2020-04-16 17:29   ` Alan Mackenzie
2020-04-16 18:01     ` Stefan Monnier
     [not found] ` <handler.40658.B.158704107627532.ack@debbugs.gnu.org>
2020-04-16 17:31   ` bug#40658: (Fforward-comment: Incorrect recognition of end of comment in C++) Alan Mackenzie

Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).