From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!not-for-mail From: Stefan Monnier Newsgroups: gmane.emacs.devel Subject: Re: `isearch-allow-scroll' - a misnomer and a bad design Date: Fri, 09 Sep 2011 20:58:42 -0400 Message-ID: References: <20110909215255.GD2733@acm.acm> <7002A9DA9A804F0B9F6F251FD3A2B263@us.oracle.com> NNTP-Posting-Host: lo.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: dough.gmane.org 1315616334 12432 80.91.229.12 (10 Sep 2011 00:58:54 GMT) X-Complaints-To: usenet@dough.gmane.org NNTP-Posting-Date: Sat, 10 Sep 2011 00:58:54 +0000 (UTC) Cc: 'Alan Mackenzie' , emacs-devel@gnu.org To: "Drew Adams" Original-X-From: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Sat Sep 10 02:58:50 2011 Return-path: Envelope-to: ged-emacs-devel@m.gmane.org Original-Received: from lists.gnu.org ([140.186.70.17]) by lo.gmane.org with esmtp (Exim 4.69) (envelope-from ) id 1R2BuD-00085m-5X for ged-emacs-devel@m.gmane.org; Sat, 10 Sep 2011 02:58:49 +0200 Original-Received: from localhost ([::1]:44062 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1R2BuC-0001gP-IW for ged-emacs-devel@m.gmane.org; Fri, 09 Sep 2011 20:58:48 -0400 Original-Received: from eggs.gnu.org ([140.186.70.92]:34334) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1R2Bu9-0001gJ-LY for emacs-devel@gnu.org; Fri, 09 Sep 2011 20:58:46 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1R2Bu8-0000MC-PO for emacs-devel@gnu.org; Fri, 09 Sep 2011 20:58:45 -0400 Original-Received: from ironport2-out.teksavvy.com ([206.248.154.183]:25996 helo=ironport2-out.pppoe.ca) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1R2Bu8-0000M8-Jy for emacs-devel@gnu.org; Fri, 09 Sep 2011 20:58:44 -0400 X-IronPort-Anti-Spam-Filtered: true X-IronPort-Anti-Spam-Result: Av0EABu1ak64rwMJ/2dsb2JhbABCqBd5gVIBAQQBViMFCws0EhQYDSSICbdahm4EoC+EQg X-IronPort-AV: E=Sophos;i="4.68,359,1312171200"; d="scan'208";a="135408105" Original-Received: from 184-175-3-9.dsl.teksavvy.com (HELO pastel.home) ([184.175.3.9]) by ironport2-out.pppoe.ca with ESMTP/TLS/ADH-AES256-SHA; 09 Sep 2011 20:58:43 -0400 Original-Received: by pastel.home (Postfix, from userid 20848) id C1DBD589E5; Fri, 9 Sep 2011 20:58:42 -0400 (EDT) In-Reply-To: <7002A9DA9A804F0B9F6F251FD3A2B263@us.oracle.com> (Drew Adams's message of "Fri, 9 Sep 2011 16:07:02 -0700") User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.50 (gnu/linux) X-detected-operating-system: by eggs.gnu.org: Genre and OS details not recognized. X-Received-From: 206.248.154.183 X-BeenThere: emacs-devel@gnu.org X-Mailman-Version: 2.1.14 Precedence: list List-Id: "Emacs development discussions." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Original-Sender: emacs-devel-bounces+ged-emacs-devel=m.gmane.org@gnu.org Xref: news.gmane.org gmane.emacs.devel:143835 Archived-At: > I meant that it is not _necessarily_ about scrolling. I understand that that > was the original motivation, and it remains an important use case (for people > who want scrolling). Actually, it's more than that: its intention is about scrolling. You can abuse it, but some people like pass-through for some set of commands but not for others, so isearch-allow-scroll is meant to control that for scrolling commands. We all agree that the fact that C-u got folded into it is definitely a misfeature in this respect. Maybe what you want is a new option `isearch-pass-through-categories', which would be a list of symbol properties, so any command who has a non-nil value for one of those properties is allowed to run without exiting isearch. Then `scroll-command' becomes one possible element of isearch-pass-through-categories. > It is true, AFAICT. Nothing prevents you from putting property > `isearch-scroll' on *any* command, to get Isearch to pass through > to it. But you still only have one boolean value to control what commands to pass through. So what would you name this boolean option? `isearch-a-few-more-commands-run-within-isearch'? What if people want pass-through for scrolling commands but not for your new command? Stefan