unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
From: "Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
To: 56234@debbugs.gnu.org
Subject: bug#56234: 29.0.50; [PATCH] Incorrect reference to signal-process-functions default value
Date: Sun, 26 Jun 2022 12:34:08 +0200	[thread overview]
Message-ID: <m1bkufd87z.fsf@yahoo.es> (raw)
In-Reply-To: m1bkufd87z.fsf.ref@yahoo.es

[-- Attachment #1: Type: text/plain, Size: 515 bytes --]

Severity: minor
Tags: patch

In Emacs 29 NEWS there is this text:

'signal-process' now consults the list 'signal-process-functions'.
[...] The hitherto existing implementation has been
moved to 'signal-default-interrupt-process'.

However, the only other reference to signal-default-interrupt-process I
can find is in the ELisp manual.  I think the correct reference is
internal-default-signal-process.  If so, it needs to be updated in the
ELisp manual and in NEWS.

I've attached a patch with the fix.  Thanks.


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-Fix-typo-in-signal-process-functions.patch --]
[-- Type: text/x-patch, Size: 1642 bytes --]

From 8e186e7d21d1048e2fb5817d9772e19a8f0b9701 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Daniel=20Mart=C3=ADn?= <mardani29@yahoo.es>
Date: Sun, 26 Jun 2022 12:27:55 +0200
Subject: [PATCH] Fix typo in signal-process-functions

* doc/lispref/processes.texi (Signals to Processes): Update reference
to correct default variable in the ELisp manual.
* etc/NEWS: The same for the NEWS entry.
---
 doc/lispref/processes.texi | 2 +-
 etc/NEWS                   | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/doc/lispref/processes.texi b/doc/lispref/processes.texi
index 9e0bd98a54..14856b9e05 100644
--- a/doc/lispref/processes.texi
+++ b/doc/lispref/processes.texi
@@ -1512,7 +1512,7 @@ Signals to Processes
 for @code{signal-process}.  These functions are called in the order of
 the list, until one of them returns non-@code{nil}.  The default
 function, which shall always be the last in this list, is
-@code{signal-default-interrupt-process}.
+@code{internal-default-signal-process}.
 
 This is the mechanism, how Tramp implements @code{signal-process}.
 @end defvar
diff --git a/etc/NEWS b/etc/NEWS
index 6c04ae164c..cefe056060 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -2840,7 +2840,7 @@ invocation.  Such shells are POSIX conformant by default.
 This is to determine which function has to be called in order to
 deliver the signal.  This allows Tramp to send the signal to remote
 asynchronous processes.  The hitherto existing implementation has been
-moved to 'signal-default-interrupt-process'.
+moved to 'internal-default-signal-process'.
 
 +++
 ** 'list-system-processes' now returns remote process IDs.
-- 
2.34.1


       reply	other threads:[~2022-06-26 10:34 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <m1bkufd87z.fsf.ref@yahoo.es>
2022-06-26 10:34 ` Daniel Martín via Bug reports for GNU Emacs, the Swiss army knife of text editors [this message]
2022-06-26 14:47   ` bug#56234: 29.0.50; [PATCH] Incorrect reference to signal-process-functions default value Lars Ingebrigtsen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=m1bkufd87z.fsf@yahoo.es \
    --to=bug-gnu-emacs@gnu.org \
    --cc=56234@debbugs.gnu.org \
    --cc=mardani29@yahoo.es \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://git.savannah.gnu.org/cgit/emacs.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).