GNU bug report logs - #61707
Remove function-alias-p NOERROR argument

Previous Next

Package: emacs;

Reported by: Mattias Engdegård <mattias.engdegard <at> gmail.com>

Date: Wed, 22 Feb 2023 13:45:01 UTC

Severity: normal

Done: Mattias Engdegård <mattias.engdegard <at> gmail.com>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 61707 in the body.
You can then email your comments to 61707 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to bug-gnu-emacs <at> gnu.org:
bug#61707; Package emacs. (Wed, 22 Feb 2023 13:45:01 GMT) Full text and rfc822 format available.

Acknowledgement sent to Mattias Engdegård <mattias.engdegard <at> gmail.com>:
New bug report received and forwarded. Copy sent to bug-gnu-emacs <at> gnu.org. (Wed, 22 Feb 2023 13:45:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Mattias Engdegård <mattias.engdegard <at> gmail.com>
To: Emacs Bug Report <bug-gnu-emacs <at> gnu.org>
Cc: Eli Zaretskii <eliz <at> gnu.org>
Subject: Remove function-alias-p NOERROR argument
Date: Wed, 22 Feb 2023 14:44:21 +0100
[Message part 1 (text/plain, inline)]
`function-alias-p` is new in Emacs 29, and takes an optional NOERROR argument which has become entirely useless in Emacs 30 where it is only present for compatibility.

Given that the NOERROR argument is not very useful in Emacs 29 either, what about removing it in that version as well? After all, rather silly to introduce something that's known to go away the next release.

Proposed patch attached.

[0001-Remove-function-alias-p-noerror-argument.patch (application/octet-stream, attachment)]

Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#61707; Package emacs. (Wed, 22 Feb 2023 14:14:02 GMT) Full text and rfc822 format available.

Message #8 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Eli Zaretskii <eliz <at> gnu.org>
To: Mattias Engdegård <mattias.engdegard <at> gmail.com>,
 Lars Ingebrigtsen <larsi <at> gnus.org>
Cc: bug-gnu-emacs <at> gnu.org
Subject: Re: Remove function-alias-p NOERROR argument
Date: Wed, 22 Feb 2023 16:13:18 +0200
> From: Mattias Engdegård <mattias.engdegard <at> gmail.com>
> Date: Wed, 22 Feb 2023 14:44:21 +0100
> Cc: Eli Zaretskii <eliz <at> gnu.org>
> 
> `function-alias-p` is new in Emacs 29, and takes an optional NOERROR argument which has become entirely useless in Emacs 30 where it is only present for compatibility.
> 
> Given that the NOERROR argument is not very useful in Emacs 29 either, what about removing it in that version as well? After all, rather silly to introduce something that's known to go away the next release.
> 
> Proposed patch attached.

Lars added this function, so it would be prudent to ask him.

My opinion is that it's too late for such changes: this function is
with us for more than a year, so I guess we will have to live with
optional argument even if it's useless in Emacs 30 and later.




Information forwarded to bug-gnu-emacs <at> gnu.org:
bug#61707; Package emacs. (Wed, 22 Feb 2023 15:27:01 GMT) Full text and rfc822 format available.

Message #11 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Lars Ingebrigtsen <larsi <at> gnus.org>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Mattias Engdegård <mattias.engdegard <at> gmail.com>,
 bug-gnu-emacs <at> gnu.org
Subject: Re: Remove function-alias-p NOERROR argument
Date: Wed, 22 Feb 2023 16:26:07 +0100
Eli Zaretskii <eliz <at> gnu.org> writes:

> Lars added this function, so it would be prudent to ask him.
>
> My opinion is that it's too late for such changes: this function is
> with us for more than a year, so I guess we will have to live with
> optional argument even if it's useless in Emacs 30 and later.

I agree with Eli.

(The optional argument has been made obsolete in Emacs 30, so it can be
removed in Emacs 33-ish.)




Reply sent to Mattias Engdegård <mattias.engdegard <at> gmail.com>:
You have taken responsibility. (Wed, 22 Feb 2023 16:44:02 GMT) Full text and rfc822 format available.

Notification sent to Mattias Engdegård <mattias.engdegard <at> gmail.com>:
bug acknowledged by developer. (Wed, 22 Feb 2023 16:44:02 GMT) Full text and rfc822 format available.

Message #16 received at 61707-done <at> debbugs.gnu.org (full text, mbox):

From: Mattias Engdegård <mattias.engdegard <at> gmail.com>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: Lars Ingebrigtsen <larsi <at> gnus.org>, 61707-done <at> debbugs.gnu.org
Subject: Re: Remove function-alias-p NOERROR argument
Date: Wed, 22 Feb 2023 17:43:39 +0100
22 feb. 2023 kl. 15.13 skrev Eli Zaretskii <eliz <at> gnu.org>:

> My opinion is that it's too late for such changes

Reasonable people may disagree, but I won't insist. Closing bug.





bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Thu, 23 Mar 2023 11:24:05 GMT) Full text and rfc822 format available.

This bug report was last modified 1 year and 34 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.