GNU bug report logs - #79117
31.0.50; Fix `image-supported-file-p' doc string, and possibly autoload cookie.

Previous Next

To reply to this bug, email your comments to 79117 AT debbugs.gnu.org.
There is no need to reopen the bug first.

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

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

From: David Ponce <da_vid <at> orange.fr>
To: bug-gnu-emacs <at> gnu.org
Subject: 31.0.50; Fix `image-supported-file-p' doc string, and possibly
 autoload cookie.
Date: Mon, 28 Jul 2025 20:46:31 +0200

[Message part 1 (text/plain, inline)]

Hello,

Please find attached a small patch proposal to fix the autoload cookie
and doc string of the function `image-supported-file-p' in image.el.
And a possible change log:

2025-07-28  David Ponce  <da_vid <at> orange.fr>

	* lisp/image.el (image-supported-file-p): Fix autoload cookie and
	doc string.

In GNU Emacs 31.0.50 (build 7, x86_64-pc-linux-gnu, GTK+ Version
 3.24.49, cairo version 1.18.2) of 2025-07-28
Repository revision: 62fba3a723d10d53a8bac1351e762874ec0c9360
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12101018
System Description: Fedora Linux 42 (KDE Plasma Desktop Edition)

Configured using:
 'configure --with-native-compilation=no
 --with-tree-sitter=no'

Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GPM GSETTINGS HARFBUZZ JPEG
LCMS2 LIBOTF LIBSELINUX LIBSYSTEMD LIBXML2 M17N_FLT MODULES NOTIFY
INOTIFY PDUMPER PNG RSVG SECCOMP SOUND SQLITE3 THREADS TIFF
TOOLKIT_SCROLL_BARS WEBP X11 XDBE XIM XINERAMA XINPUT2 XPM XRANDR GTK3
ZLIB

Important settings:
  value of $LC_TIME: fr_FR.utf8
  value of $LANG: fr_FR.UTF-8
  locale-coding-system: utf-8-unix


Thanks

[image.el-doc-fix-V0.patch (text/x-patch, attachment)]

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: David Ponce <da_vid <at> orange.fr>
Cc: 79117-done <at> debbugs.gnu.org
Subject: Re: bug#79117: 31.0.50;
 Fix `image-supported-file-p' doc string, and possibly autoload cookie.
Date: Tue, 29 Jul 2025 14:09:53 +0300

> Date: Mon, 28 Jul 2025 20:46:31 +0200
> From:  David Ponce via "Bug reports for GNU Emacs,
>  the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
> 
> Please find attached a small patch proposal to fix the autoload cookie
> and doc string of the function `image-supported-file-p' in image.el.
> And a possible change log:
> 
> 2025-07-28  David Ponce  <da_vid <at> orange.fr>
> 
> 	* lisp/image.el (image-supported-file-p): Fix autoload cookie and
> 	doc string.

Thanks.  That doc string had other wording problems, so I've fixed it
a bit differently (on the emacs-30 release branch).  I'm therefore
closing the bug.

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

From: Da_vid <da_vid <at> orange.fr>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 79117-done <at> debbugs.gnu.org
Subject: Re: bug#79117: 31.0.50; Fix `image-supported-file-p' doc string, and possibly autoload cookie.
Date: Tue, 29 Jul 2025 13:23:18 +0200

Le 29 juillet 2025 13:09:53 GMT+02:00, Eli Zaretskii <eliz <at> gnu.org> a écrit :
>> Date: Mon, 28 Jul 2025 20:46:31 +0200
>> From:  David Ponce via "Bug reports for GNU Emacs,
>>  the Swiss army knife of text editors" <bug-gnu-emacs <at> gnu.org>
>> 
>> Please find attached a small patch proposal to fix the autoload cookie
>> and doc string of the function `image-supported-file-p' in image.el.
>> And a possible change log:
>> 
>> 2025-07-28  David Ponce  <da_vid <at> orange.fr>
>> 
>> 	* lisp/image.el (image-supported-file-p): Fix autoload cookie and
>> 	doc string.
>
>Thanks.  That doc string had other wording problems, so I've fixed it
>a bit differently (on the emacs-30 release branch).  I'm therefore
>closing the bug.

Thank you!
So I guess it is expected to have the autoload cookie slightly indented?

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

From: Eli Zaretskii <eliz <at> gnu.org>
To: Da_vid <da_vid <at> orange.fr>
Cc: 79117 <at> debbugs.gnu.org
Subject: Re: bug#79117: 31.0.50;
 Fix `image-supported-file-p' doc string, and possibly autoload cookie.
Date: Tue, 29 Jul 2025 14:33:36 +0300

> Date: Tue, 29 Jul 2025 13:23:18 +0200
> From: Da_vid <da_vid <at> orange.fr>
> CC: 79117-done <at> debbugs.gnu.org
> 
> So I guess it is expected to have the autoload cookie slightly indented?

image.el is preloaded, so its autoload cookies are no-ops.

Is it really important to remove that indentation?

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

From: Da_vid <da_vid <at> orange.fr>
To: Eli Zaretskii <eliz <at> gnu.org>
Cc: 79117 <at> debbugs.gnu.org
Subject: Re: bug#79117: 31.0.50; Fix `image-supported-file-p' doc string, and possibly autoload cookie.
Date: Tue, 29 Jul 2025 13:40:09 +0200

Le 29 juillet 2025 13:33:36 GMT+02:00, Eli Zaretskii <eliz <at> gnu.org> a écrit :
>> Date: Tue, 29 Jul 2025 13:23:18 +0200
>> From: Da_vid <da_vid <at> orange.fr>
>> CC: 79117-done <at> debbugs.gnu.org
>> 
>> So I guess it is expected to have the autoload cookie slightly indented?
>
>image.el is preloaded, so its autoload cookies are no-ops.
>
>Is it really important to remove that indentation?

Ah, OK, make sense.

No, it is not important, I was just curious about why this one is not aligned like are others in this file ;-)

Previous Next

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