From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED.blaine.gmane.org!not-for-mail From: =?UTF-8?Q?Jo=C3=A3o_?= =?UTF-8?Q?T=C3=A1vora?= Newsgroups: gmane.emacs.bugs Subject: bug#35254: 27.0.50; cc-mode/electric-pair-mode/electric-layout-mode: bad trailing whitespace behavior in cc-mode Date: Tue, 14 May 2019 11:34:24 +0100 Message-ID: References: <87ftqms9db.fsf@secretsauce.net> <871s15k7ll.fsf@gmail.com> <20190513195323.GB5525@ACM> <20190514092735.GB4231@ACM> Mime-Version: 1.0 Content-Type: multipart/alternative; boundary="00000000000047291d0588d69463" Injection-Info: blaine.gmane.org; posting-host="blaine.gmane.org:195.159.176.226"; logging-data="12679"; mail-complaints-to="usenet@blaine.gmane.org" Cc: Noam Postavsky , Dima Kogan , 35254@debbugs.gnu.org, Stefan Monnier To: Alan Mackenzie Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Tue May 14 12:47:38 2019 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([209.51.188.17]) by blaine.gmane.org with esmtps (TLS1.0:RSA_AES_256_CBC_SHA1:256) (Exim 4.89) (envelope-from ) id 1hQUy9-0003BP-JF for geb-bug-gnu-emacs@m.gmane.org; Tue, 14 May 2019 12:47:37 +0200 Original-Received: from localhost ([127.0.0.1]:45266 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hQUy8-0001Wg-FV for geb-bug-gnu-emacs@m.gmane.org; Tue, 14 May 2019 06:47:36 -0400 Original-Received: from eggs.gnu.org ([209.51.188.92]:34866) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1hQUqz-0004Gq-JE for bug-gnu-emacs@gnu.org; Tue, 14 May 2019 06:40:14 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1hQUly-0005fQ-7u for bug-gnu-emacs@gnu.org; Tue, 14 May 2019 06:35:07 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:33949) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1hQUly-0005eA-3I; Tue, 14 May 2019 06:35:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1hQUlx-0008QP-RB; Tue, 14 May 2019 06:35:01 -0400 X-Loop: help-debbugs@gnu.org Resent-From: =?UTF-8?Q?Jo=C3=A3o_?= =?UTF-8?Q?T=C3=A1vora?= Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org, bug-cc-mode@gnu.org Resent-Date: Tue, 14 May 2019 10:35:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 35254 X-GNU-PR-Package: emacs,cc-mode Original-Received: via spool by 35254-submit@debbugs.gnu.org id=B35254.155783008632355 (code B ref 35254); Tue, 14 May 2019 10:35:01 +0000 Original-Received: (at 35254) by debbugs.gnu.org; 14 May 2019 10:34:46 +0000 Original-Received: from localhost ([127.0.0.1]:47493 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hQUlh-0008Pn-P2 for submit@debbugs.gnu.org; Tue, 14 May 2019 06:34:46 -0400 Original-Received: from mail-io1-f51.google.com ([209.85.166.51]:46031) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1hQUle-0008PW-EG for 35254@debbugs.gnu.org; Tue, 14 May 2019 06:34:44 -0400 Original-Received: by mail-io1-f51.google.com with SMTP id b3so12578294iob.12 for <35254@debbugs.gnu.org>; Tue, 14 May 2019 03:34:42 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=ztzhQui25WfP+zO7daMePSYFbW/3WRjx/4x9K6btk7M=; b=o+rTm32y4vnFfLMt9doOYy5PaUAAc7WcepYyWMGzrlRoBl16sKi7WpPsexDvXhH76u pwIMlm5l03oHAVD9vOxrK3Vy16r5QYNgBfkTdEuLj+Vk5NYh7ZmrZkWpCaMorQDzHR4M Fi4hZNymlMmPs77hZm36eNPZhWs1lCDUXcytSq8heHY4ebT95E+auj5OaOouqyh/6yXB WVqIRIaAs5wqjKX7rtAtNmLQByd6E04YTNPz5Hf0+2Etp97t2xiiw6bulUfTSUvyPDbJ sxLLcGlJ0V6h1kaiMFK/azEx8F6/lLCLGIOHM6l3jTBNqy01O5Dtizu7ZKtXyG02me4s BfWw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=ztzhQui25WfP+zO7daMePSYFbW/3WRjx/4x9K6btk7M=; b=eq4Ivw+fmTKBFwazde/+ajPDHd7+PGjhQap64A0CmI6q03arQBVm7+xGm5mf1ZJbrs Z54cCIXAEMdLtIKyY9Gq6U0FfpWphBRtASaU0FD3R6Q5bov1sv+fo20rLkxiEWTy6h76 AdW/nUJ+jRVEqpZoqjmNNwNMJo7o74uls2yLDn1pGV3o9T8AUj1K7Nb7CnMGUPfepTbj RTxwSpfYM1NIKs95HF3lH5dC/KRk59VzVQGYecwCQtIQgw3mq2BJE/riHRMAkfbEsQOy 4nmCgvc0YwDJBilZvXntFkT2OWSlrY1C7R/ikEs6P38u5GvacP0h2hs8ZosNrcfyy0CQ jHnw== X-Gm-Message-State: APjAAAXFMS7NIuwdgwWmTA0d/UbIYUpINNeTvgp8Czg8PxNmjxfTOhnx boXnAbg24M2YtYtQzq2P3rnTrBywBbI8vFPZiLU= X-Google-Smtp-Source: APXvYqyUFp+4iFL7ygMMCKYlVpaZ9vuVK7oCFhyvfOhvWBwHrN8RQQHw1pDkGjXqwM85GkO5kzyJVFYAUZWIDaPJkLU= X-Received: by 2002:a5d:9e0f:: with SMTP id h15mr13169567ioh.48.1557830075622; Tue, 14 May 2019 03:34:35 -0700 (PDT) In-Reply-To: <20190514092735.GB4231@ACM> X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 2.2.x-3.x [generic] X-Received-From: 209.51.188.43 X-BeenThere: bug-gnu-emacs@gnu.org List-Id: "Bug reports for GNU Emacs, the Swiss army knife of text editors" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.org gmane.emacs.bugs:159267 Archived-At: --00000000000047291d0588d69463 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable On Tue, May 14, 2019 at 10:27 AM Alan Mackenzie wrote: > The bug is, type lots of s in a row; the indentation WS isn't > getting removed from the blank lines. Currently electric-indent-inhibit > is inhibiting this removal. > Do you mean the "removal of the WS in the lines preceding the current". In other words, do you mean "removal of the trailing WS that was once proper indentation"? Or do you think that the current line, the one where point stands, should not be indented at all in certain electric-* variable combinations and or c-electric-* variable? Which of those combinations? > Probably. Maybe Jo=C3=A3o should check this, once he's fully back with u= s. > I'm afraid I can't put a date on that. There's a bun in the oven... An important development towards figuring out this issue is that a significant fraction of us agrees on what the behavior should be in what cases. Then we should code tests that assert that behavior possibly reusing the fixtures in electric-tests.el. > The same bug occurs in Python Mode. > Succinctly, the bug is that on pressing lots of times in a row, the > indentation WS is being left on the blank lines rather than being > removed. I see. That does make sense. But, to be sure, we _dont_ what to remove the indentation WS on the "current" line, right? Jo=C3=A3o --00000000000047291d0588d69463 Content-Type: text/html; charset="UTF-8" Content-Transfer-Encoding: quoted-printable
On Tue, May 14, 2019 at 10:27 AM Alan Mac= kenzie <acm@muc.de> wrote:
The bug is, type lots of <CR>s in a row; the indentation WS isn't=
getting removed from the blank lines.=C2=A0 Currently electric-indent-inhib= it
is inhibiting this removal.

Do you mean= the "removal of the WS in the lines preceding the current".
=
In other words, do you mean "removal of the trailing WS tha= t was once
proper indentation"?=C2=A0
Or do you think that the current line, the one where point sta= nds, should
not be indented at all in certain electric-* var= iable combinations and or
c-electric-* variable?=C2=A0 Which= of those combinations?
=C2=A0
Probably.=C2=A0 Maybe Jo=C3=A3o should check this, once he's fully back= with us.

I'm afraid I can't pu= t a date on that. There's a bun in the oven...

=
An important development towards figuring out this issue is that a
significant fraction of us agrees on what the behavior should b= e
in what cases.=C2=A0 Then we should code tests that assert= that behavior
possibly reusing the fixtures in electric-tests.el= .

> The same bug occurs in Python Mode.
>= Succinctly, the bug is that on pressing <CR> lots of times in a row,= the
> indentation WS is being left on the blank lines rather than be= ing
> removed.

I see.=C2= =A0 That does make sense. But, to be sure, we _dont_ what to
remove the indentation WS on the "cur= rent" line, right?

<= /span>
Jo=C3=A3o

--00000000000047291d0588d69463--