From mboxrd@z Thu Jan 1 00:00:00 1970 Path: news.gmane.io!.POSTED.blaine.gmane.org!not-for-mail From: Simen =?UTF-8?Q?Heggest=C3=B8yl?= via "Bug reports for GNU Emacs, the Swiss army knife of text editors" Newsgroups: gmane.emacs.bugs Subject: bug#67000: 30.0.50; [PATCH] Add support for reading/writing IELM input history Date: Thu, 04 Jan 2024 08:46:09 +0100 Message-ID: <87a5pl1qym.fsf@runbox.com> References: <87sf5ge0v6.fsf@runbox.com> Reply-To: Simen =?UTF-8?Q?Heggest=C3=B8yl?= Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="=-=-=" Injection-Info: ciao.gmane.io; posting-host="blaine.gmane.org:116.202.254.214"; logging-data="37931"; mail-complaints-to="usenet@ciao.gmane.io" User-Agent: Gnus/5.13 (Gnus v5.13) Cc: 67000@debbugs.gnu.org To: Stefan Kangas Original-X-From: bug-gnu-emacs-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Thu Jan 04 08:47:28 2024 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 1rLIRs-0009bq-2u for geb-bug-gnu-emacs@m.gmane-mx.org; Thu, 04 Jan 2024 08:47:28 +0100 Original-Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1rLIRY-0002Ki-JE; Thu, 04 Jan 2024 02:47:09 -0500 Original-Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1rLIRS-0002Ih-1t for bug-gnu-emacs@gnu.org; Thu, 04 Jan 2024 02:47:04 -0500 Original-Received: from debbugs.gnu.org ([2001:470:142:5::43]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.90_1) (envelope-from ) id 1rLIRP-0007uC-6I for bug-gnu-emacs@gnu.org; Thu, 04 Jan 2024 02:47:00 -0500 Original-Received: from Debian-debbugs by debbugs.gnu.org with local (Exim 4.84_2) (envelope-from ) id 1rLIRS-0006er-85 for bug-gnu-emacs@gnu.org; Thu, 04 Jan 2024 02:47:02 -0500 X-Loop: help-debbugs@gnu.org Resent-From: Simen =?UTF-8?Q?Heggest=C3=B8yl?= Original-Sender: "Debbugs-submit" Resent-CC: bug-gnu-emacs@gnu.org Resent-Date: Thu, 04 Jan 2024 07:47:02 +0000 Resent-Message-ID: Resent-Sender: help-debbugs@gnu.org X-GNU-PR-Message: followup 67000 X-GNU-PR-Package: emacs X-GNU-PR-Keywords: patch Original-Received: via spool by 67000-submit@debbugs.gnu.org id=B67000.170435439325296 (code B ref 67000); Thu, 04 Jan 2024 07:47:02 +0000 Original-Received: (at 67000) by debbugs.gnu.org; 4 Jan 2024 07:46:33 +0000 Original-Received: from localhost ([127.0.0.1]:53564 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1rLIQy-0006Ys-IH for submit@debbugs.gnu.org; Thu, 04 Jan 2024 02:46:33 -0500 Original-Received: from mailtransmit04.runbox.com ([2a0c:5a00:149::25]:35306) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1rLIQp-0006Lc-M6 for 67000@debbugs.gnu.org; Thu, 04 Jan 2024 02:46:31 -0500 Original-Received: from mailtransmit02.runbox ([10.9.9.162] helo=aibo.runbox.com) by mailtransmit04.runbox.com with esmtps (TLS1.2) tls TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256 (Exim 4.93) (envelope-from ) id 1rLIQd-001mln-Nm for 67000@debbugs.gnu.org; Thu, 04 Jan 2024 08:46:11 +0100 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=runbox.com; s=selector1; h=Content-Type:MIME-Version:Message-ID:Date:References: In-Reply-To:Subject:Cc:To:From; bh=HISgdEG8zPpUYEtQY+VtWR7KGfoCmFUR+AHxvdo/iro=; b=VpNIVM7sDgNGbfm4IkZ8W6LxWP Hqux+Q+QcLljcfwZIONwsv0MLEsJ8BQOsfNkQ96+4v3DE2TIeetY54dwQGjP4GH5PCJPjAPQZDEjE tUQeaqYckq8iZusce52r1fXMYBpBxnPOfDeVlpRgCDC0Ff9q9bMdE/426Y7saE4M2jP0C8uMIsMht wIMb+gZ2nQcGLAWg8pYNHmokD4s0vfVhfR7ajKdyLYvIgz6trf+hRERVD5YEz4gY9jiWYXZj7Z6Nl 38A37f0u69r3o3Dh3mMj4D05/H2TF0lcqyt+qQenF9d6N6wkUZDE9jCPcFlZSvPfDCc456dgVONLE 6oAaiCTw==; Original-Received: from [10.9.9.74] (helo=submission03.runbox) by mailtransmit02.runbox with esmtp (Exim 4.86_2) (envelope-from ) id 1rLIQd-0005zX-7d; Thu, 04 Jan 2024 08:46:11 +0100 Original-Received: by submission03.runbox with esmtpsa [Authenticated ID (963757)] (TLS1.2:ECDHE_SECP256R1__RSA_SHA256__AES_256_GCM:256) (Exim 4.93) id 1rLIQc-00AWxF-6n; Thu, 04 Jan 2024 08:46:10 +0100 In-Reply-To: (Stefan Kangas's message of "Thu, 14 Dec 2023 17:06:10 -0800") 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-bounces+geb-bug-gnu-emacs=m.gmane-mx.org@gnu.org Xref: news.gmane.io gmane.emacs.bugs:277298 Archived-At: --=-=-= Content-Type: text/plain Hi Stefan, thank you for your response. Stefan Kangas writes: > - This should be called out in NEWS. Yup, done in the revised patch attached. > - Does it need any other documentation changes? I don't think so; IELM is only mentioned briefly in the Emacs manual. > - Do we usually provide options to disable this stuff? Not as far as I can tell. >> +(defvar ielm--history-file-name >> + (locate-user-emacs-file "ielm-history.el") > > Shouldn't that be .eld? Hm, the file will contain actual Lisp expressions, so I'd think that .el is fitting? -- Simen --=-=-= Content-Type: text/x-diff Content-Disposition: attachment; filename=0001-Add-support-for-reading-writing-IELM-input-history.patch >From 3ac85190a90fd85bb7a960cd4895c3bd4e41b085 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Simen=20Heggest=C3=B8yl?= Date: Wed, 8 Nov 2023 16:08:08 +0100 Subject: [PATCH] Add support for reading/writing IELM input history * lisp/ielm.el (inferior-emacs-lisp-mode): Add support for reading/writing input history. (ielm--history-file-name): Name of the file to read/write IELM input history. --- etc/NEWS | 5 +++++ lisp/ielm.el | 12 ++++++++++++ 2 files changed, 17 insertions(+) diff --git a/etc/NEWS b/etc/NEWS index 1cdb12c3958..f65d53e4827 100644 --- a/etc/NEWS +++ b/etc/NEWS @@ -1216,6 +1216,11 @@ chat buffers use by default. *** New command 'customize-dirlocals'. This command pops up a buffer to edit the settings in ".dir-locals.el". +** IELM + +--- +*** IELM now remembers input history between sessions. + * New Modes and Packages in Emacs 30.1 diff --git a/lisp/ielm.el b/lisp/ielm.el index 777aebb70cf..46a2a1799da 100644 --- a/lisp/ielm.el +++ b/lisp/ielm.el @@ -152,6 +152,10 @@ ielm-header \\[describe-mode] for help.\n" "Message to display when IELM is started.") +(defvar ielm--history-file-name + (locate-user-emacs-file "ielm-history.el") + "Name of the file to read/write IELM input history.") + (defvaralias 'inferior-emacs-lisp-mode-map 'ielm-map) (defvar-keymap ielm-map :doc "Keymap for IELM mode." @@ -605,6 +609,11 @@ inferior-emacs-lisp-mode #'ielm-indirect-setup-hook 'append t) (setq comint-indirect-setup-function #'emacs-lisp-mode) + ;; Input history + (setq-local comint-input-ring-file-name ielm--history-file-name) + (setq-local kill-buffer-hook #'comint-write-input-ring) + (comint-read-input-ring t) + ;; A dummy process to keep comint happy. It will never get any input (unless (comint-check-proc (current-buffer)) ;; Was cat, but on non-Unix platforms that might not exist, so @@ -631,6 +640,9 @@ inferior-emacs-lisp-mode (set-marker comint-last-input-start (ielm-pm)) (set-process-filter (get-buffer-process (current-buffer)) 'comint-output-filter))) +(unless noninteractive + (add-hook 'kill-emacs-hook #'comint-write-input-ring)) + (defun ielm-get-old-input nil ;; Return the previous input surrounding point (save-excursion -- 2.39.2 --=-=-=--