unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: Alan Mackenzie <acm@muc.de>
To: Paul Eggert <eggert@cs.ucla.edu>
Cc: 22884@debbugs.gnu.org
Subject: bug#22884: 25.0.92; C/l mode editing takes waaaayy too long
Date: Fri, 4 Mar 2016 14:47:59 +0000	[thread overview]
Message-ID: <20160304144759.GB2117@acm.fritz.box> (raw)
In-Reply-To: <56D8CC45.2090102@cs.ucla.edu>

Hello, Paul.

On Thu, Mar 03, 2016 at 03:44:05PM -0800, Paul Eggert wrote:
> On 03/03/2016 12:51 PM, Eli Zaretskii wrote:
> > I'd rather we fixed CC Mode instead of "fixing" the
> > sources it chokes on.

> Me too, but I don't see a straightforward cc-mode fix for this.

> Even if we come up with a fix, for some time we'll have the problem of 
> people using old versions of Emacs to look at the new Emacs source code. 
> And until there's a fix Emacs is reeeaally sluggish when editing 
> config.h at least. So I'd rather fix at least config.h, and (since I've 
> already done the work) I'm also mildly inclined to fix the other files 
> to be consistent, as in the attached patch (which also covers licenses 
> in the other .c and .h files).

Thanks for the (new) patch.  I've tried it out, and it appears to build
and run just fine.

I have had an idea for fixing Emacs so that we don't have this problem
with parens in column 0.  That is only to scan comments in the forward
direction, and to mark them with text properties.  `back_comment' will
then be little more than checking these text properties are up to date,
and then doing a backward text property search.

> From c7c5fa7e01492963aab142d91b00cb872cb49686 Mon Sep 17 00:00:00 2001
> From: Paul Eggert <eggert@cs.ucla.edu>
> Date: Thu, 3 Mar 2016 15:42:28 -0800
> Subject: [PATCH] Rework C source files to avoid ^(
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit

> Work around Bug#22884 by rewording comments and strings to avoid ‘(’
> at the start of a line unless it starts a function.  Although this
> change is a hack and we should fix cc-mode’s performance for C files
> that have ‘(’ at the start of a line in a comment or string, the
> change does fix the immediate problem.
> ---

-- 
Alan Mackenzie (Nuremberg, Germany).





  reply	other threads:[~2016-03-04 14:47 UTC|newest]

Thread overview: 34+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-03-02 18:08 bug#22884: 25.0.92; C/l mode editing takes waaaayy too long Paul Eggert
2016-03-03 12:49 ` Alan Mackenzie
2016-03-03 17:54   ` Paul Eggert
2016-03-03 19:23     ` Alan Mackenzie
2016-03-03 20:38       ` Eli Zaretskii
2016-03-03 21:57       ` Paul Eggert
2016-03-03 22:59         ` Alan Mackenzie
2016-03-03 20:51     ` Eli Zaretskii
2016-03-03 23:44       ` Paul Eggert
2016-03-04 14:47         ` Alan Mackenzie [this message]
2016-03-04 20:32           ` Paul Eggert
2016-03-04 21:08             ` Alan Mackenzie
2016-03-13 10:03               ` Alan Mackenzie
2016-03-08 14:02             ` Alan Mackenzie
2016-03-09  8:25               ` Paul Eggert
2016-03-09  9:28                 ` John Wiegley
2016-03-09  9:37                   ` Paul Eggert
2016-03-09 21:30                     ` John Wiegley
2016-03-09 10:56                   ` Alan Mackenzie
2016-03-09 14:44                     ` Drew Adams
2016-03-09 17:04                       ` Alan Mackenzie
2016-03-09 17:14                         ` Drew Adams
2016-03-03 20:40   ` Eli Zaretskii
2016-03-03 22:27     ` Alan Mackenzie
2016-03-03 23:18     ` Alan Mackenzie
2016-03-04  8:32       ` Eli Zaretskii
2016-03-04  9:37         ` Alan Mackenzie
2016-03-15  3:07 ` Stefan Monnier
2016-05-08 23:10   ` Dmitry Gutov
2016-05-13 19:35 ` Paul Eggert
2016-05-13 20:37   ` Glenn Morris
2016-05-13 21:09     ` Paul Eggert
2022-04-28 11:22   ` Lars Ingebrigtsen
2022-04-28 19:35     ` Paul Eggert

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160304144759.GB2117@acm.fritz.box \
    --to=acm@muc.de \
    --cc=22884@debbugs.gnu.org \
    --cc=eggert@cs.ucla.edu \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).