unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
@ 2024-06-05 13:16 Philippe Schnoebelen via Bug reports for GNU Emacs, the Swiss army knife of text editors
  2024-06-05 16:56 ` Mattias Engdegård
  0 siblings, 1 reply; 19+ messages in thread
From: Philippe Schnoebelen via Bug reports for GNU Emacs, the Swiss army knife of text editors @ 2024-06-05 13:16 UTC (permalink / raw)
  To: 71379

The following example code has been put in my ~/foo.el

(defun check-value (v)
   ;; Let's demonstrate different treament of formats by elisp compiler
   (cond (v
	 (print (format "Congratulations! v=%S is non-NIL."))
	 ;; oops, we forgot to pass v in `format's &rest args
	 t)
	(t ;; otherwise
	 (user-error "Try again. v=%S is null")
	 ;; same error, here when calling `user-error'
	 nil)))


When I byte-compile it with M-x byte-compile-file I get the following 
warning message:

      In check-value:
      foo.el:4:18: Warning: ‘format’ called with 0 args to fill 1 format 
field(s)

which is very useful.

But why don't I get a similar error message for my call to user-error ? 
It would be useful too.

Note that when I evaluate (check-value nil) I get an error with the 
following *Backtrace*, showing that my call to
`user-error' fails for exactly the same reason that evaluating 
`(check-value t)' fails.

Debugger entered--Lisp error: (error "Not enough arguments for format 
string")
   format-message("Try again. v=%S is null")
   apply(format-message "Try again. v=%S is null" nil)
   user-error("Try again. v=%S is null")
   (if v (print (format "Congratulations! v=%S is non-NIL.")) 
(user-error "Try again. v=%S is null"))
   check-value(nil)
   (progn (check-value nil))
   eval((progn (check-value nil)) t)
   elisp--eval-last-sexp(t)
   eval-last-sexp(t)
   eval-print-last-sexp(nil)
   funcall-interactively(eval-print-last-sexp nil)
   call-interactively(eval-print-last-sexp nil nil)

I thought you'd like to know.

Thanks for your time and dedication in developing GNU Emacs !

--philippe

In GNU Emacs 29.3 (build 1, aarch64-apple-darwin21.6.0, NS 
appkit-2113.60 Version 12.6.6 (Build 21G646)) of
  2024-03-24 built on armbob.lan
Windowing system distributor 'Apple', version 10.3.2487
System Description:  macOS 14.5

Configured using:
  'configure --with-ns '--enable-locallisppath=/Library/Application
  Support/Emacs/${version}/site-lisp:/Library/Application 
Support/Emacs/site-lisp' --with-modules
  'CFLAGS=-DFD_SETSIZE=10000 -DDARWIN_UNLIMITED_SELECT' --with-x-toolkit=no'





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-05 13:16 bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments Philippe Schnoebelen via Bug reports for GNU Emacs, the Swiss army knife of text editors
@ 2024-06-05 16:56 ` Mattias Engdegård
  2024-06-05 17:48   ` Eli Zaretskii
  0 siblings, 1 reply; 19+ messages in thread
From: Mattias Engdegård @ 2024-06-05 16:56 UTC (permalink / raw)
  To: philippe schnoebelen; +Cc: 71379, Eli Zaretskii

I stumbled on this report by accident:

> But why don't I get a similar error message for my call to user-error ?

That's a clear oversight, thank you for reporting it. Will fix.

Should be trivial enough for master (and I have made a full bootstrap) but I'm asking Eli for confirmation just in case.







^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-05 16:56 ` Mattias Engdegård
@ 2024-06-05 17:48   ` Eli Zaretskii
  2024-06-05 19:20     ` Mattias Engdegård
  0 siblings, 1 reply; 19+ messages in thread
From: Eli Zaretskii @ 2024-06-05 17:48 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: 71379, schnoebelen.ph

> From: Mattias Engdegård <mattias.engdegard@gmail.com>
> Date: Wed, 5 Jun 2024 18:56:52 +0200
> Cc: 71379@debbugs.gnu.org,
>  Eli Zaretskii <eliz@gnu.org>
> 
> I stumbled on this report by accident:
> 
> > But why don't I get a similar error message for my call to user-error ?
> 
> That's a clear oversight, thank you for reporting it. Will fix.
> 
> Should be trivial enough for master (and I have made a full bootstrap) but I'm asking Eli for confirmation just in case.

Thanks, I'd like to see the changes before making up my mind about
that.  The problem is definitely not serious, so the fix should be
safe enough to qualify.





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-05 17:48   ` Eli Zaretskii
@ 2024-06-05 19:20     ` Mattias Engdegård
  2024-06-06  4:38       ` Eli Zaretskii
  0 siblings, 1 reply; 19+ messages in thread
From: Mattias Engdegård @ 2024-06-05 19:20 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 71379, schnoebelen.ph

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

5 juni 2024 kl. 19.48 skrev Eli Zaretskii <eliz@gnu.org>:

> Thanks, I'd like to see the changes before making up my mind about
> that.  The problem is definitely not serious, so the fix should be
> safe enough to qualify.

Sorry, meant to attach the patch.


[-- Attachment #2: 0001-Format-string-warnings-for-more-functions-bug-71379.patch --]
[-- Type: application/octet-stream, Size: 1098 bytes --]

From 6f758e82a37a6b62abc71459c8878de7217834ae Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Mattias=20Engdeg=C3=A5rd?= <mattiase@acm.org>
Date: Wed, 5 Jun 2024 18:28:59 +0200
Subject: [PATCH] Format string warnings for more functions (bug#71379)

* lisp/emacs-lisp/bytecomp.el (byte-compile-format-like):
Add message-box, message-or-box, warn and user-error.
---
 lisp/emacs-lisp/bytecomp.el | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/lisp/emacs-lisp/bytecomp.el b/lisp/emacs-lisp/bytecomp.el
index 03cfbe6f4c9..d59e89bd7de 100644
--- a/lisp/emacs-lisp/bytecomp.el
+++ b/lisp/emacs-lisp/bytecomp.el
@@ -1643,7 +1643,8 @@ byte-compile-format-warn
          nargs (if (= nargs 1) "" "s")
          nfields (if (= nfields 1) "" "s"))))))
 
-(dolist (elt '(format message format-message error))
+(dolist (elt '( format message format-message message-box message-or-box
+                warn error user-error))
   (put elt 'byte-compile-format-like t))
 
 ;; Warn if the function or macro is being redefined with a different
-- 
2.32.0 (Apple Git-132)


^ permalink raw reply related	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-05 19:20     ` Mattias Engdegård
@ 2024-06-06  4:38       ` Eli Zaretskii
  2024-06-06  8:49         ` Mattias Engdegård
  0 siblings, 1 reply; 19+ messages in thread
From: Eli Zaretskii @ 2024-06-06  4:38 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: 71379, schnoebelen.ph

> From: Mattias Engdegård <mattias.engdegard@gmail.com>
> Date: Wed, 5 Jun 2024 21:20:53 +0200
> Cc: schnoebelen.ph@gmail.com,
>  71379@debbugs.gnu.org
> 
> 5 juni 2024 kl. 19.48 skrev Eli Zaretskii <eliz@gnu.org>:
> 
> > Thanks, I'd like to see the changes before making up my mind about
> > that.  The problem is definitely not serious, so the fix should be
> > safe enough to qualify.
> 
> Sorry, meant to attach the patch.

On second thought, this is still fine to go in now, but I wonder
whether we should move these to the places where the corresponding
functions are defined, and leave in bytecomp.c only those which are
primitives defined in C?  That would mean 'warn' and 'user-error'
should be moved to their respective Lisp files.  WDYT?





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-06  4:38       ` Eli Zaretskii
@ 2024-06-06  8:49         ` Mattias Engdegård
  2024-06-06 23:22           ` Stefan Kangas
  0 siblings, 1 reply; 19+ messages in thread
From: Mattias Engdegård @ 2024-06-06  8:49 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 71379-done, philippe schnoebelen

6 juni 2024 kl. 06.38 skrev Eli Zaretskii <eliz@gnu.org>:

> On second thought, this is still fine to go in now, but I wonder
> whether we should move these to the places where the corresponding
> functions are defined, and leave in bytecomp.c only those which are
> primitives defined in C?  That would mean 'warn' and 'user-error'
> should be moved to their respective Lisp files.

Indeed it's a question I always ask myself every so often, and sometimes we distribute properties to definitions. 
However, in this case it seems better to keep them in one place: it's only a small handful of functions, and this way the properties are kept local to the byte-compiler.

Thank you, the change is now on master. Closing.






^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-06  8:49         ` Mattias Engdegård
@ 2024-06-06 23:22           ` Stefan Kangas
  2024-06-07  6:21             ` Eli Zaretskii
  2024-06-07  9:32             ` Mattias Engdegård
  0 siblings, 2 replies; 19+ messages in thread
From: Stefan Kangas @ 2024-06-06 23:22 UTC (permalink / raw)
  To: Mattias Engdegård, Eli Zaretskii; +Cc: 71379, philippe schnoebelen

Mattias Engdegård <mattias.engdegard@gmail.com> writes:

> 6 juni 2024 kl. 06.38 skrev Eli Zaretskii <eliz@gnu.org>:
>
>> On second thought, this is still fine to go in now, but I wonder
>> whether we should move these to the places where the corresponding
>> functions are defined, and leave in bytecomp.c only those which are
>> primitives defined in C?  That would mean 'warn' and 'user-error'
>> should be moved to their respective Lisp files.
>
> Indeed it's a question I always ask myself every so often, and sometimes we distribute properties to definitions.
> However, in this case it seems better to keep them in one place: it's only a small handful of functions, and this way the properties are kept local to the byte-compiler.

No strong opinion, but I think this cuts both ways: keeping it local to
the definition of the functions themselves makes it clear to anyone
looking there for inspiration how to add such warnings to their own
`format` functions.





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-06 23:22           ` Stefan Kangas
@ 2024-06-07  6:21             ` Eli Zaretskii
  2024-06-07 13:55               ` Andrea Corallo
  2024-06-07  9:32             ` Mattias Engdegård
  1 sibling, 1 reply; 19+ messages in thread
From: Eli Zaretskii @ 2024-06-07  6:21 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 71379, schnoebelen.ph, mattias.engdegard

> From: Stefan Kangas <stefankangas@gmail.com>
> Date: Fri, 7 Jun 2024 01:22:49 +0200
> Cc: 71379@debbugs.gnu.org, philippe schnoebelen <schnoebelen.ph@gmail.com>
> 
> Mattias Engdegård <mattias.engdegard@gmail.com> writes:
> 
> > 6 juni 2024 kl. 06.38 skrev Eli Zaretskii <eliz@gnu.org>:
> >
> >> On second thought, this is still fine to go in now, but I wonder
> >> whether we should move these to the places where the corresponding
> >> functions are defined, and leave in bytecomp.c only those which are
> >> primitives defined in C?  That would mean 'warn' and 'user-error'
> >> should be moved to their respective Lisp files.
> >
> > Indeed it's a question I always ask myself every so often, and sometimes we distribute properties to definitions.
> > However, in this case it seems better to keep them in one place: it's only a small handful of functions, and this way the properties are kept local to the byte-compiler.
> 
> No strong opinion, but I think this cuts both ways: keeping it local to
> the definition of the functions themselves makes it clear to anyone
> looking there for inspiration how to add such warnings to their own
> `format` functions.

Indeed.  In addition, some functions marked with this property already
do that in their own files.  Which was why I asked that question in
the first place.





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-06 23:22           ` Stefan Kangas
  2024-06-07  6:21             ` Eli Zaretskii
@ 2024-06-07  9:32             ` Mattias Engdegård
  1 sibling, 0 replies; 19+ messages in thread
From: Mattias Engdegård @ 2024-06-07  9:32 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 71379, philippe schnoebelen, Eli Zaretskii

7 juni 2024 kl. 01.22 skrev Stefan Kangas <stefankangas@gmail.com>:

> No strong opinion, but I think this cuts both ways: keeping it local to
> the definition of the functions themselves makes it clear to anyone
> looking there for inspiration how to add such warnings to their own
> `format` functions.

Yes, it's a matter of judgement which has to be reconsidered from time to time. I see no reason for doing anything about this right now, but this may change in the future. The reasons for not distributing function properties can be quite strong as well.






^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-07  6:21             ` Eli Zaretskii
@ 2024-06-07 13:55               ` Andrea Corallo
  2024-06-07 15:50                 ` Eli Zaretskii
  0 siblings, 1 reply; 19+ messages in thread
From: Andrea Corallo @ 2024-06-07 13:55 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 71379, schnoebelen.ph, mattias.engdegard, Stefan Kangas

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Stefan Kangas <stefankangas@gmail.com>
>> Date: Fri, 7 Jun 2024 01:22:49 +0200
>> Cc: 71379@debbugs.gnu.org, philippe schnoebelen <schnoebelen.ph@gmail.com>
>> 
>> Mattias Engdegård <mattias.engdegard@gmail.com> writes:
>> 
>> > 6 juni 2024 kl. 06.38 skrev Eli Zaretskii <eliz@gnu.org>:
>> >
>> >> On second thought, this is still fine to go in now, but I wonder
>> >> whether we should move these to the places where the corresponding
>> >> functions are defined, and leave in bytecomp.c only those which are
>> >> primitives defined in C?  That would mean 'warn' and 'user-error'
>> >> should be moved to their respective Lisp files.
>> >
>> > Indeed it's a question I always ask myself every so often, and
>> > sometimes we distribute properties to definitions.
>> > However, in this case it seems better to keep them in one place:
>> > it's only a small handful of functions, and this way the
>> > properties are kept local to the byte-compiler.
>> 
>> No strong opinion, but I think this cuts both ways: keeping it local to
>> the definition of the functions themselves makes it clear to anyone
>> looking there for inspiration how to add such warnings to their own
>> `format` functions.
>
> Indeed.  In addition, some functions marked with this property already
> do that in their own files.  Which was why I asked that question in
> the first place.

Agreed.  Maybe in the future even worth having a declare for it?  We
have them already for things like ex 'important-return-value'.

  Andrea





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-07 13:55               ` Andrea Corallo
@ 2024-06-07 15:50                 ` Eli Zaretskii
  2024-06-08 18:15                   ` Andrea Corallo
  0 siblings, 1 reply; 19+ messages in thread
From: Eli Zaretskii @ 2024-06-07 15:50 UTC (permalink / raw)
  To: Andrea Corallo; +Cc: 71379, schnoebelen.ph, mattias.engdegard, stefankangas

> From: Andrea Corallo <acorallo@gnu.org>
> Cc: Stefan Kangas <stefankangas@gmail.com>,  71379@debbugs.gnu.org,
>   schnoebelen.ph@gmail.com,  mattias.engdegard@gmail.com
> Date: Fri, 07 Jun 2024 09:55:08 -0400
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> >> No strong opinion, but I think this cuts both ways: keeping it local to
> >> the definition of the functions themselves makes it clear to anyone
> >> looking there for inspiration how to add such warnings to their own
> >> `format` functions.
> >
> > Indeed.  In addition, some functions marked with this property already
> > do that in their own files.  Which was why I asked that question in
> > the first place.
> 
> Agreed.  Maybe in the future even worth having a declare for it?  We
> have them already for things like ex 'important-return-value'.

Yes, could be a good idea.





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-07 15:50                 ` Eli Zaretskii
@ 2024-06-08 18:15                   ` Andrea Corallo
  2024-06-08 18:18                     ` Mattias Engdegård
  0 siblings, 1 reply; 19+ messages in thread
From: Andrea Corallo @ 2024-06-08 18:15 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 71379, schnoebelen.ph, mattias.engdegard, stefankangas

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Andrea Corallo <acorallo@gnu.org>
>> Cc: Stefan Kangas <stefankangas@gmail.com>,  71379@debbugs.gnu.org,
>>   schnoebelen.ph@gmail.com,  mattias.engdegard@gmail.com
>> Date: Fri, 07 Jun 2024 09:55:08 -0400
>> 
>> Eli Zaretskii <eliz@gnu.org> writes:
>> 
>> >> No strong opinion, but I think this cuts both ways: keeping it local to
>> >> the definition of the functions themselves makes it clear to anyone
>> >> looking there for inspiration how to add such warnings to their own
>> >> `format` functions.
>> >
>> > Indeed.  In addition, some functions marked with this property already
>> > do that in their own files.  Which was why I asked that question in
>> > the first place.
>> 
>> Agreed.  Maybe in the future even worth having a declare for it?  We
>> have them already for things like ex 'important-return-value'.
>
> Yes, could be a good idea.

Okay, will do after we branch emacs30.

  Andrea





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-08 18:15                   ` Andrea Corallo
@ 2024-06-08 18:18                     ` Mattias Engdegård
  2024-06-08 18:32                       ` Eli Zaretskii
  0 siblings, 1 reply; 19+ messages in thread
From: Mattias Engdegård @ 2024-06-08 18:18 UTC (permalink / raw)
  To: Andrea Corallo; +Cc: 71379, schnoebelen.ph, Eli Zaretskii, stefankangas

8 juni 2024 kl. 20.15 skrev Andrea Corallo <acorallo@gnu.org>:

>>> Agreed.  Maybe in the future even worth having a declare for it?  We
>>> have them already for things like ex 'important-return-value'.
>> 
>> Yes, could be a good idea.
> 
> Okay, will do after we branch emacs30.

Again, I'm not very fond of the idea (or I would have done it long ago myself).






^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-08 18:18                     ` Mattias Engdegård
@ 2024-06-08 18:32                       ` Eli Zaretskii
  2024-06-09 13:57                         ` Mattias Engdegård
  0 siblings, 1 reply; 19+ messages in thread
From: Eli Zaretskii @ 2024-06-08 18:32 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: 71379, schnoebelen.ph, acorallo, stefankangas

> From: Mattias Engdegård <mattias.engdegard@gmail.com>
> Date: Sat, 8 Jun 2024 20:18:09 +0200
> Cc: Eli Zaretskii <eliz@gnu.org>,
>  stefankangas@gmail.com,
>  71379@debbugs.gnu.org,
>  schnoebelen.ph@gmail.com
> 
> 8 juni 2024 kl. 20.15 skrev Andrea Corallo <acorallo@gnu.org>:
> 
> >>> Agreed.  Maybe in the future even worth having a declare for it?  We
> >>> have them already for things like ex 'important-return-value'.
> >> 
> >> Yes, could be a good idea.
> > 
> > Okay, will do after we branch emacs30.
> 
> Again, I'm not very fond of the idea (or I would have done it long ago myself).

Why not?  Without a rationale, how can we discuss this?





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-08 18:32                       ` Eli Zaretskii
@ 2024-06-09 13:57                         ` Mattias Engdegård
  2024-06-10  8:11                           ` Andrea Corallo
  0 siblings, 1 reply; 19+ messages in thread
From: Mattias Engdegård @ 2024-06-09 13:57 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: 71379, schnoebelen.ph, acorallo, stefankangas

8 juni 2024 kl. 20.32 skrev Eli Zaretskii <eliz@gnu.org>:

> Why not?

As I said, I think the drawbacks outweigh the advantages right now.
It's much easier to keep track of the functions that are subject to the warning when they are gathered in a single place, and there is an advantage in keeping the mechanism local.






^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-09 13:57                         ` Mattias Engdegård
@ 2024-06-10  8:11                           ` Andrea Corallo
  2024-06-10 14:27                             ` Mattias Engdegård
  0 siblings, 1 reply; 19+ messages in thread
From: Andrea Corallo @ 2024-06-10  8:11 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: 71379, schnoebelen.ph, Eli Zaretskii, stefankangas

Mattias Engdegård <mattias.engdegard@gmail.com> writes:

> 8 juni 2024 kl. 20.32 skrev Eli Zaretskii <eliz@gnu.org>:
>
>> Why not?
>
> As I said, I think the drawbacks outweigh the advantages right now.
> It's much easier to keep track of the functions that are subject to the warning when they are gathered in a single place, and there is an advantage in keeping the mechanism local.

But this is not local only already, see 'info-xref-output' and
'info-xref-output-error'.  There might be other occurencies outside
emacs.git as well.

  Andrea





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-10  8:11                           ` Andrea Corallo
@ 2024-06-10 14:27                             ` Mattias Engdegård
  2024-06-10 21:14                               ` Andrea Corallo
  0 siblings, 1 reply; 19+ messages in thread
From: Mattias Engdegård @ 2024-06-10 14:27 UTC (permalink / raw)
  To: Andrea Corallo; +Cc: 71379, schnoebelen.ph, Eli Zaretskii, stefankangas

10 juni 2024 kl. 10.11 skrev Andrea Corallo <acorallo@gnu.org>:

> But this is not local only already, see 'info-xref-output' and
> 'info-xref-output-error'.

Very much aware of that, and don't think it's worth doing anything about.

>  There might be other occurencies outside
> emacs.git as well.

Already checked that as well, and there were almost none -- all I could find was a single occurrence in one package. Not that we really care since it's an internal and undocumented interface.






^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-10 14:27                             ` Mattias Engdegård
@ 2024-06-10 21:14                               ` Andrea Corallo
  2024-06-11 14:30                                 ` Mattias Engdegård
  0 siblings, 1 reply; 19+ messages in thread
From: Andrea Corallo @ 2024-06-10 21:14 UTC (permalink / raw)
  To: Mattias Engdegård; +Cc: 71379, schnoebelen.ph, Eli Zaretskii, stefankangas

Mattias Engdegård <mattias.engdegard@gmail.com> writes:

> 10 juni 2024 kl. 10.11 skrev Andrea Corallo <acorallo@gnu.org>:
>
>> But this is not local only already, see 'info-xref-output' and
>> 'info-xref-output-error'.
>
> Very much aware of that, and don't think it's worth doing anything about.
>
>>  There might be other occurencies outside
>> emacs.git as well.
>
> Already checked that as well, and there were almost none -- all I
> could find was a single occurrence in one package. Not that we really
> care since it's an internal and undocumented interface.

You can indeed have your opinions, but again this shows that the
mechanism is not local only already, and that it's already in use
outside as there was probably need for it.





^ permalink raw reply	[flat|nested] 19+ messages in thread

* bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments
  2024-06-10 21:14                               ` Andrea Corallo
@ 2024-06-11 14:30                                 ` Mattias Engdegård
  0 siblings, 0 replies; 19+ messages in thread
From: Mattias Engdegård @ 2024-06-11 14:30 UTC (permalink / raw)
  To: Andrea Corallo; +Cc: 71379, schnoebelen.ph, Eli Zaretskii, stefankangas

10 juni 2024 kl. 23.14 skrev Andrea Corallo <acorallo@gnu.org>:

> You can indeed have your opinions, but again this shows that the
> mechanism is not local only already, and that it's already in use
> outside as there was probably need for it.

No it doesn't.

I don't know what is special about this warning, enough for you to argue strongly about it, but maybe you don't either -- once we take a strong position about something, even by accident, it's difficult to stop, and to admit that it's perhaps not that important after all.

So let's give it a rest. I will keep revisiting this again, as I have many times before, and of course so may you. If we do something, let's make sure we know why, not just because we have made up our minds about it.






^ permalink raw reply	[flat|nested] 19+ messages in thread

end of thread, other threads:[~2024-06-11 14:30 UTC | newest]

Thread overview: 19+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-06-05 13:16 bug#71379: 29.3; Elisp compiler: warnings for formats given insufficiently many arguments Philippe Schnoebelen via Bug reports for GNU Emacs, the Swiss army knife of text editors
2024-06-05 16:56 ` Mattias Engdegård
2024-06-05 17:48   ` Eli Zaretskii
2024-06-05 19:20     ` Mattias Engdegård
2024-06-06  4:38       ` Eli Zaretskii
2024-06-06  8:49         ` Mattias Engdegård
2024-06-06 23:22           ` Stefan Kangas
2024-06-07  6:21             ` Eli Zaretskii
2024-06-07 13:55               ` Andrea Corallo
2024-06-07 15:50                 ` Eli Zaretskii
2024-06-08 18:15                   ` Andrea Corallo
2024-06-08 18:18                     ` Mattias Engdegård
2024-06-08 18:32                       ` Eli Zaretskii
2024-06-09 13:57                         ` Mattias Engdegård
2024-06-10  8:11                           ` Andrea Corallo
2024-06-10 14:27                             ` Mattias Engdegård
2024-06-10 21:14                               ` Andrea Corallo
2024-06-11 14:30                                 ` Mattias Engdegård
2024-06-07  9:32             ` Mattias Engdegård

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).