From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Dmitry Gutov Newsgroups: gmane.emacs.bugs Subject: bug#16116: 24.3.50; smie-indent-close aligns inner closing paren with the outer opening paren Date: Sat, 14 Dec 2013 17:04:47 +0200 Message-ID: <87y53niicw.fsf@yandex.ru> References: <87sity96he.fsf@yandex.ru> <6B783955-B72E-4876-BCF2-C864F3D74A62@sanityinc.com> NNTP-Posting-Host: plane.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: ger.gmane.org 1387033519 23856 80.91.229.3 (14 Dec 2013 15:05:19 GMT) X-Complaints-To: usenet@ger.gmane.org NNTP-Posting-Date: Sat, 14 Dec 2013 15:05:19 +0000 (UTC) Cc: 16116@debbugs.gnu.org, Steve Purcell To: Stefan Monnier Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Sat Dec 14 16:05:22 2013 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by plane.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1VrqmQ-0002ag-5H for geb-bug-gnu-emacs@m.gmane.org; Sat, 14 Dec 2013 16:05:22 +0100 Original-Received: from localhost ([::1]:47606 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VrqmP-0000br-Gs for geb-bug-gnu-emacs@m.gmane.org; Sat, 14 Dec 2013 10:05:21 -0500 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:58889) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VrqmI-0000bh-0h for bug-gnu-emacs@gnu.org; Sat, 14 Dec 2013 10:05:19 -0500 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1VrqmC-0007Hg-Fc for bug-gnu-emacs@gnu.org; Sat, 14 Dec 2013 10:05:13 -0500 Original-Received: from debbugs.gnu.org ([140.186.70.43]:36144) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1VrqmC-0007Hc-DO for bug-gnu-emacs@gnu.org; Sat, 14 Dec 2013 10:05:08 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.80) (envelope-from ) id 1Vrqm6-0006LC-6K for bug-gnu-emacs@gnu.org; Sat, 14 Dec 2013 10:05:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Dmitry Gutov Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Sat, 14 Dec 2013 15:05:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 16116 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 16116-submit@debbugs.gnu.org id=B16116.138703350024366 (code B ref 16116); Sat, 14 Dec 2013 15:05:02 +0000 Original-Received: (at 16116) by debbugs.gnu.org; 14 Dec 2013 15:05:00 +0000 Original-Received: from localhost ([127.0.0.1]:50163 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1Vrqm3-0006Ku-D2 for submit@debbugs.gnu.org; Sat, 14 Dec 2013 10:04:59 -0500 Original-Received: from mail-ea0-f178.google.com ([209.85.215.178]:41618) by debbugs.gnu.org with esmtp (Exim 4.80) (envelope-from ) id 1Vrqm0-0006Kl-Lc for 16116@debbugs.gnu.org; Sat, 14 Dec 2013 10:04:57 -0500 Original-Received: by mail-ea0-f178.google.com with SMTP id d10so1423570eaj.9 for <16116@debbugs.gnu.org>; Sat, 14 Dec 2013 07:04:55 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20120113; h=sender:from:to:cc:subject:references:date:in-reply-to:message-id :user-agent:mime-version:content-type; bh=KzExouGUy9XglcXhNnJccz783dDIuwOC5rI0Ztt1rYM=; b=QTrU+Q8vvEqsZYBUcGVhqYv7IyVRa8JBTnkSVbhpisb3KA5HtJ6FVJXjye2xv7eC9+ qS27ye/KiZAs2rkG8QcKW7sU+p5bIptDyaGYY8GRT51Wgi0uAhs3V0QrAnDSY8OkVUJI bzJ2i7G9GXiGDFCAJMOOVqbfG3GjNzuvGdAt0BrVuZ1M758NwSOLE9ShjRyvDJggXaR0 xbdZbriw7v/Cd8kTU1wKekUqkUxRHfsuhpwG8lV87eJYHrKdZ+aYzPeECEf/o9w0zJPr TLix+WHz45b0DUj3qUiAKlWEBeseIVDfIGqtyCcgnmosia4qyM8hbsyvz8it7sbWmxXp ir8g== X-Received: by 10.14.47.130 with SMTP id t2mr8149826eeb.12.1387033495590; Sat, 14 Dec 2013 07:04:55 -0800 (PST) Original-Received: from axl ([83.168.26.131]) by mx.google.com with ESMTPSA id o1sm19405561eea.10.2013.12.14.07.04.53 for (version=TLSv1.2 cipher=RC4-SHA bits=128/128); Sat, 14 Dec 2013 07:04:54 -0800 (PST) In-Reply-To: (Stefan Monnier's message of "Sat, 14 Dec 2013 09:13:11 -0500") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3.50 (gnu/linux) X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.15 Precedence: list X-detected-operating-system: by eggs.gnu.org: GNU/Linux 3.x X-Received-From: 140.186.70.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-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.bugs:81958 Archived-At: Stefan Monnier writes: > No need to convince anyone here. I was just pointing out that what is > right will depend Sorry, I didn't get that. > so the right fix is to let the rules-function > control it, rather than to replace one hard-coded choice with another. That won't work. ruby-smie-rules is never called because smie-indent-close doesn't use smie-indent--rule, and it goes before the functions that do in smie-indent-functions. Unless you're suggesting to change either of those points. I thought rather to add a defvar to switch between the two behaviors in smie-indent-close, or make smie-indent-functions buffer-local and replace smie-indent-close there with a modified implementation. Still inclined toward the latter.