From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.org!.POSTED!not-for-mail From: Lars Ingebrigtsen Newsgroups: gmane.emacs.bugs Subject: bug#30212: 26.0.91; [PATCH] push mark before moving point in shell output Date: Tue, 17 Apr 2018 22:24:05 +0200 Message-ID: <87lgdlh996.fsf@mouse.gnus.org> References: <87bmhleki3.fsf@gmail.com> NNTP-Posting-Host: blaine.gmane.org Mime-Version: 1.0 Content-Type: text/plain X-Trace: blaine.gmane.org 1523996590 1715 195.159.176.226 (17 Apr 2018 20:23:10 GMT) X-Complaints-To: usenet@blaine.gmane.org NNTP-Posting-Date: Tue, 17 Apr 2018 20:23:10 +0000 (UTC) User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/27.0.50 (gnu/linux) Cc: 30212@debbugs.gnu.org To: Alex Branham Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane.org@gnu.org Tue Apr 17 22:23:06 2018 Return-path: Envelope-to: geb-bug-gnu-emacs@m.gmane.org Original-Received: from lists.gnu.org ([208.118.235.17]) by blaine.gmane.org with esmtp (Exim 4.84_2) (envelope-from ) id 1f8X85-0000KV-Gx for geb-bug-gnu-emacs@m.gmane.org; Tue, 17 Apr 2018 22:23:05 +0200 Original-Received: from localhost ([::1]:60819 helo=lists.gnu.org) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1f8XAC-0005sr-68 for geb-bug-gnu-emacs@m.gmane.org; Tue, 17 Apr 2018 16:25:16 -0400 Original-Received: from eggs.gnu.org ([2001:4830:134:3::10]:43119) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1f8XA3-0005on-Q6 for bug-gnu-emacs@gnu.org; Tue, 17 Apr 2018 16:25:08 -0400 Original-Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1f8XA0-0004xu-Kd for bug-gnu-emacs@gnu.org; Tue, 17 Apr 2018 16:25:07 -0400 Original-Received: from debbugs.gnu.org ([208.118.235.43]:50055) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1f8XA0-0004v8-H4 for bug-gnu-emacs@gnu.org; Tue, 17 Apr 2018 16:25:04 -0400 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1f8X9y-0000cD-Cy for bug-gnu-emacs@gnu.org; Tue, 17 Apr 2018 16:25:04 -0400 X-Loop: help-debbugs@gnu.org Resent-From: Lars Ingebrigtsen Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Tue, 17 Apr 2018 20:25:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 30212 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 30212-submit@debbugs.gnu.org id=B30212.15239966562291 (code B ref 30212); Tue, 17 Apr 2018 20:25:02 +0000 Original-Received: (at 30212) by debbugs.gnu.org; 17 Apr 2018 20:24:16 +0000 Original-Received: from localhost ([127.0.0.1]:57949 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1f8X9E-0000as-9d for submit@debbugs.gnu.org; Tue, 17 Apr 2018 16:24:16 -0400 Original-Received: from hermes.netfonds.no ([80.91.224.195]:38514) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1f8X9C-0000ak-Bk for 30212@debbugs.gnu.org; Tue, 17 Apr 2018 16:24:14 -0400 Original-Received: from 46.67.12.60.tmi.telenormobil.no ([46.67.12.60] helo=corrigan) by hermes.netfonds.no with esmtpsa (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.84_2) (envelope-from ) id 1f8X99-0006S8-8u; Tue, 17 Apr 2018 22:24:13 +0200 Original-Received: from larsi by corrigan with local (Exim 4.89) (envelope-from ) id 1f8X93-0002h5-B6; Tue, 17 Apr 2018 22:24:05 +0200 In-Reply-To: <87bmhleki3.fsf@gmail.com> (Alex Branham's message of "Mon, 22 Jan 2018 13:55:32 -0600") 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: 208.118.235.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:145520 Archived-At: Alex Branham writes: > Since shell commands move point, I find it useful to push the mark so > that I can navigate around different output using C-u C-SPC. This > patch adds the push-mark call that's necessary to > shell-command--save-pos-or-erase. [...] > (when pos > + (push-mark pos t) > (goto-char (point-max)) Hm... I don't think it would be a good idea to set the mark unexpectedly -- people have certain expectations on what commands set the mark, and setting it for every command in shell mode would be unusual and get in the way of people's work habits, I think. -- (domestic pets only, the antidote for overdose, milk.) bloggy blog: http://lars.ingebrigtsen.no