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: Thu, 3 Mar 2016 12:49:10 +0000	[thread overview]
Message-ID: <20160303124910.GA2852@acm.fritz.box> (raw)
In-Reply-To: <56D72C35.4090708@cs.ucla.edu>

Hello, Paul.

On Wed, Mar 02, 2016 at 10:08:53AM -0800, Paul Eggert wrote:
> I've been noticing this problem for a bit and figured it'd get fixed but 
> it hasn't so here is a bug report.

> With the Emacs 25 pretests, it takes waaaaayyy too long to edit some C 
> code. To reproduce the problem use the attached file (taken from the 
> Emacs source code) and run:

> emacs -Q config.h
> M-x goto-line RET 1661 RET / /

> On my six-year-old desktop the second '/' takes about 10 seconds to 
> echo.  This sort of thing makes Emacs effectively unusable for editing 
> config.h.

The problem is in config.h.  At line 14, inside a comment, appears the
following string:
    "(at your option) any later version."
.  The open paren is at column zero, so the fancy code in syntax.c then
fails to recognise the comment as a comment.  CC Mode is then
effectively communicating across the continent between L1661 and L14 by
carrier pigeon in the belief that there is non-syntactic-ws code at L14.
(Syntactic whitespace includes comments and preprocessor constructs.)

Inserting a backslash at the beginning of L14 solves the problem, as
does setting open-paren-in-column-0-is-defun-start to nil.

The next problem is that there are around 324 occurrences of "(" at
column zero in the src directory, and quite a few in lib and lib-src.
Most of them are in comments, some of them are parameter lists, and some
of them (e.g. in lisp.h) are wierd constructs of some sort.  These
contravene GNU coding standards and really need sorting out.

> In GNU Emacs 25.0.92.1 (x86_64-unknown-linux-gnu, GTK+ Version 3.18.7)
>   of 2016-03-02 built on penguin.cs.ucla.edu
> Repository revision: 100346aa226e4eacc56f390c099bb9aab585b5f4
> Windowing system distributor 'Fedora Project', version 11.0.11800000
> Configured using:
>   'configure --enable-gcc-warnings'

-- 
Alan Mackenzie (Nuremberg, Germany).





  reply	other threads:[~2016-03-03 12:49 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 [this message]
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
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=20160303124910.GA2852@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).