bug#73098: setopt float warning unexpected

2024-09-11 Thread Stefan Kangas
Stefan Monnier via "Bug reports for GNU Emacs, the Swiss army knife of text editors" writes: > I tend to agree. If the type doesn't accept the value, you can use > something lower-level than `setopt`, while you argue with the maintainer > to try and get them to change their type. > > IMO, the wh

bug#66509: 29.1.50; let-alist should support numeric indexing

2024-09-10 Thread Stefan Kangas
Artur, do you have any thoughts on the below patch? Spencer Baugh writes: > From 28e52a343f72991cafd23fea910cc5f64ac5 Mon Sep 17 00:00:00 2001 > From: Spencer Baugh > Date: Thu, 12 Oct 2023 18:01:46 -0400 > Subject: [PATCH] Support numeric indexing in let-alist > > let-alist is very useful.

bug#69097: [PATCH] Add 'kill-region-or-word' command

2024-09-02 Thread Stefan Kangas
Eli Zaretskii writes: >> From: Philip Kaludercic >> Cc: Sean Whitton , j...@linkov.net, r...@gnu.org, >> 69...@debbugs.gnu.org >> Date: Sun, 01 Sep 2024 20:06:00 + >> >> Ping. I am not sure how to proceed on this patch. IIRC the issue was >> that the notion of a word differs in Emacs a

bug#72929: [Norbert Koch] Error building emacs after 73277a4097b

2024-09-01 Thread Stefan Kangas
Norbert Koch writes: > Thanks, this does the trick for me as well. Thanks, closing.

bug#72929: [Norbert Koch] Error building emacs after 73277a4097b

2024-09-01 Thread Stefan Kangas
ld details as per `M-x report-emacs-bug`? Start of forwarded message From: Norbert Koch To: Stefan Kangas Subject: Error building emacs after 73277a4097b Date: Sun, 01 Sep 2024 16:14:22 +0200 Hello Stefan, sorry to disturb you, but I get an error when buil

bug#72245: [PATCH] Fix integer overflow when reading XPM

2024-09-01 Thread Stefan Kangas
Version: 31.1 Stefan Kangas writes: > Since we don't have an alternative patch, I will install the one I > proposed in the next couple of days. Thanks. Pushed to master as commit 73277a4097b. Closing.

bug#71646: 29.3; pixel-scroll-precision-mode overrides paging behaviour even when pixel-scroll-precision-interpolate-page is off

2024-09-01 Thread Stefan Kangas
Eli Zaretskii writes: > Ping! Should I close this? Shouldn't we rather fix the bug described by Mike? I.e. this: > Setting `pixel-scroll-precision-interpolate-page’ is supposed to > turn off the paging animation (which it does) however even when it’s > off, and invoke `cua-scroll-up’ & `cua-

bug#72453: 29.4; "use-package" macro does not load all expected related files

2024-09-01 Thread Stefan Kangas
Mattias Engdegård writes: > It seems that 644d2a83 and 8f4ad4d7 cause failures in use-package-tests on > master (log attached); the tests pass after reverting those commits. Thanks, this should now be fixed.

bug#72915: Docstrings of add-hook and remove-hook improvement?

2024-09-01 Thread Stefan Kangas
Stefan Kangas writes: > The wording in the manual is probably all that is needed here: > > This function removes FUNCTION from the hook variable HOOK. BTW. Here's another idea: If HOOK is a list of functions, remove FUNCTION from that list. If HOOK is equal to FUNC

bug#72915: Docstrings of add-hook and remove-hook improvement?

2024-09-01 Thread Stefan Kangas
Eli Zaretskii writes: >> From: Tomas Nordin >> Date: Sat, 31 Aug 2024 12:36:22 + >> >> The following on top of emacs-30. >> >> diff --git a/lisp/subr.el b/lisp/subr.el >> index 28ba30f584e..e60c4119c60 100644 >> --- a/lisp/subr.el >> +++ b/lisp/subr.el >> @@ -2090,6 +2090,10 @@ add-hook >>

bug#72757: No symbols in etc/NEWS

2024-08-31 Thread Stefan Kangas
Eli Zaretskii writes: > I made that change, but please note that it is not enough to show each > symbol in NEWS as the default when you type "C-h o". This is because > "C-h o" tests symbol-at-point for being bound, either as a function or > a variable or a face etc., and if the corresponding lib

bug#72915: Docstrings of add-hook and remove-hook improvement?

2024-08-31 Thread Stefan Kangas
Tomas Nordin writes: > diff --git a/lisp/subr.el b/lisp/subr.el > index 28ba30f584e..e60c4119c60 100644 > --- a/lisp/subr.el > +++ b/lisp/subr.el > @@ -2090,6 +2090,10 @@ add-hook >"Add to the value of HOOK the function FUNCTION. > FUNCTION is not added if already present. > > +HOOK should b

bug#72829: describe-function NEWS* scraper override

2024-08-31 Thread Stefan Kangas
Mattias Engdegård writes: > `describe-function` sometimes gives incorrect information about when > certain functions were first introduced. NEWS.unknown can be used to > fix some problems but it only works in one direction and is unable to > help when a function name appears too early. > > A robu

bug#70007: [PATCH] native JSON encoder

2024-08-31 Thread Stefan Kangas
Stefan Monnier writes: >> And against the additional variable to make this more >> backward-compatible? > > Yup. The var would be my second-best choice (and I assume it's > immediately declared obsolete). I tend to agree with Stefan M here.