From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Lars Ingebrigtsen Newsgroups: gmane.emacs.bugs Subject: bug#33301: 27.0.50; broken elisp indentation for non-definition symbols starting with "def.." Date: Sun, 23 Aug 2020 14:26:46 +0200 Message-ID: <87o8n1fszd.fsf@gnus.org> References: <87zhukh1ri.fsf@gmail.com> <87tvksv21u.fsf@web.de> <87efbvrj4c.fsf@gmail.com> <8736sbumzj.fsf@web.de> <87a7mjqdym.fsf@gmail.com> <877ehnowgy.fsf@web.de> <875zx6r5a3.fsf@gmail.com> <87efbumvd0.fsf@web.de> <87r2fuh1tm.fsf@gmail.com> <871s7uqbvf.fsf@gmail.com> <871rjyloc4.fsf@gnus.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="8133"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/28.0.50 (gnu/linux) Cc: Michael Heerdegen , 33301@debbugs.gnu.org, Noam Postavsky To: =?UTF-8?Q?Jo=C3=A3o_?= =?UTF-8?Q?T=C3=A1vora?= Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Sun Aug 23 14:27:11 2020 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane-mx.org Original-Received: from lists.gnu.org ([209.51.188.17]) by ciao.gmane.io with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1k9p5b-00020w-2a for geb-bug-gnu-emacs@m.gmane-mx.org; Sun, 23 Aug 2020 14:27:11 +0200 Original-Received: from localhost ([::1]:54986 helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1k9p5a-0005sl-4T for geb-bug-gnu-emacs@m.gmane-mx.org; Sun, 23 Aug 2020 08:27:10 -0400 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]:56308) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1k9p5S-0005sV-Vg for bug-gnu-emacs@gnu.org; Sun, 23 Aug 2020 08:27:03 -0400 Original-Received: from debbugs.gnu.org ([209.51.188.43]:40694) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1k9p5S-0000Fd-33 for bug-gnu-emacs@gnu.org; Sun, 23 Aug 2020 08:27:02 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1k9p5S-0006CW-0H for bug-gnu-emacs@gnu.org; Sun, 23 Aug 2020 08:27:02 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Lars Ingebrigtsen Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sun, 23 Aug 2020 12:27:01 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 33301 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: confirmed Original-Received: via spool by 33301-submit@debbugs.gnu.org id=B33301.159818561923827 (code B ref 33301); Sun, 23 Aug 2020 12:27:01 +0000 Original-Received: (at 33301) by debbugs.gnu.org; 23 Aug 2020 12:26:59 +0000 Original-Received: from localhost ([127.0.0.1]:52240 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1k9p5O-0006CF-Oa for submit@debbugs.gnu.org; Sun, 23 Aug 2020 08:26:58 -0400 Original-Received: from quimby.gnus.org ([95.216.78.240]:46062) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1k9p5M-0006Bz-Qn for 33301@debbugs.gnu.org; Sun, 23 Aug 2020 08:26:57 -0400 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=gnus.org; s=20200322; h=Content-Transfer-Encoding:Content-Type:MIME-Version:Message-ID :In-Reply-To:Date:References:Subject:Cc:To:From:Sender:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Id:List-Help:List-Unsubscribe:List-Subscribe: List-Post:List-Owner:List-Archive; bh=Ek8vB1Mf2MWLJzBKiiYuzNVMKFNJzUiAHp77SNIwlXw=; b=KO2znXtmcTu+m3z6x75uSUZ6sI UVRnaYssxLf7SQ/OsQxeT5G4P9huu3SO/YY03zmKDAo/qA1BZbqkibsV2D7KlyZClj4F58T2zB8MZ q6qIoVDL3xXBmwvmaoI0PpIMLUt8aIUiwbbTe+z51VlAZYBCVUgRMUuKB6Ab0Vekva74=; Original-Received: from cm-84.212.202.86.getinternet.no ([84.212.202.86] helo=xo) by quimby with esmtpsa (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1k9p5E-0006OX-2t; Sun, 23 Aug 2020 14:26:50 +0200 Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAABGdBTUEAALGPC/xhBQAAACBj SFJNAAB6JgAAgIQAAPoAAACA6AAAdTAAAOpgAAA6mAAAF3CculE8AAAAD1BMVEUeFBRcMCGqWTDZ lGD////llHMYAAAAAWJLR0QEj2jZUQAAAAd0SU1FB+QIFwwVKcCiUM8AAAG9SURBVDjLbVSLscQg CBQrAGwgYAMv0n9vb1GT3I/J3CSufHaBK6UUSeOSRkIi5YfhhvovgIjryT+R0sZvQJp8ATOrd/vM 3iI9PManQwQn0M/1We/oEccEbOVwETwqSuuk953Cuw+HGTVKoJ2vXN2FKTQBeimWplI1uuHjm121 dizuHzzq8LxtGe3NrfoE8jrTm0puD/DmEkeWIDhUsec+r5xgytTd7mPZlYJW0b7quIvnSak4BFbb bXvhCqIRG0jiKwc06JAoVnZywRCQYBSQwlBAXYC6WGrJmaGGx7GKQTY1zZ4wYjakGDbLoWFueZp+ ANImUF0XkH6I6yOCVmuaWvZEJzs008cWS3XOcs42Wh19/E0Aoi3aQPE+XOOQBWzHCXT0v69qmejY AgCoYTrK5MfaeVeR4veRBCeAaa3rDQcg28saeIkTI3IBdYq/Qocz5OJL5rsJFH6qzzHMn2c5CUXU VGN2PpW/PY78tCSCp54PMITUJyCD6AGgGtqZ4cGCwff2sCJrPgs143uoyUzQ9r0uh5V2DagfJNeE UbtkgGFhKJm3XGQd90izWsqaa5ux3O6t2jdq5F7S/vf5B1KKMiP0fugcAAAAJXRFWHRkYXRlOmNy ZWF0ZQAyMDIwLTA4LTIzVDEyOjIxOjQxKzAwOjAwkAnczgAAACV0RVh0ZGF0ZTptb2RpZnkAMjAy MC0wOC0yM1QxMjoyMTo0MSswMDowMOFUZHIAAAAASUVORK5CYII= X-Now-Playing: The Stooges's _The Stooges_: "I Wanna Be Your Dog" In-Reply-To: ("=?UTF-8?Q?Jo=C3=A3o_?= =?UTF-8?Q?T=C3=A1vora?="'s message of "Sat, 22 Aug 2020 17:19:25 +0100") X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list 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-mx.org@gnu.org Original-Sender: "bug-gnu-emacs" Xref: news.gmane.io gmane.emacs.bugs:186043 Archived-At: Jo=C3=A3o T=C3=A1vora writes: > I wouldn't count 34 as "oodles" and don't think a new line for each > occurrence of what is essentially a breach of convention is a high > price to pay. Even converting some of those to macros or "make-foo" > could be worth it if it would enable non-surprising indentation. Changing functions to macros, or renaming functions from def* to make*, just because we have a slightly odd heuristic in Emacs Lisp mode doesn't sound quite right to me. > As for the problem of needing to load macros before indenting forms > where they appears, that's already very much a thing. We wouldn't be > creating new problems there, it's just the way it is. That's true, but it would exacerbate the problem. But the main problem is that it would indent the code differently than it does now, and that leads to whitespace churn in the vc, which we should avoid unless we have a very, very good reason not to. And this just doesn't seem like a good enough reason... > As for out-of-tree definitions, we could be lenient and have this > saner indentation be controlled by a variable which we would default > to 'insane, but to 'sane inside Emacs's source, via directory local > variables. I'd be against that -- again, because it leads to whitespace VC churn. --=20 (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no