unofficial mirror of bug-gnu-emacs@gnu.org 
 help / color / mirror / code / Atom feed
* bug#58373: Support image-type `jpg'
@ 2022-10-08  1:43 Thuna
  2022-10-08 10:36 ` Stefan Kangas
  0 siblings, 1 reply; 3+ messages in thread
From: Thuna @ 2022-10-08  1:43 UTC (permalink / raw)
  To: 58373

Currently, `jpg' is not in image-types, despite `jpeg' being in it.
Similarly, image descriptors of the form `(image :type jpg...)' do not
display properly, while those of the form `(image :type jpeg...)' do.

Changing the type from jpg to jpeg (for a jpg image) does display the
image, so it's not like Emacs is incapable of doing it; I imagine this
should be a trivial fix.





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

* bug#58373: Support image-type `jpg'
  2022-10-08  1:43 bug#58373: Support image-type `jpg' Thuna
@ 2022-10-08 10:36 ` Stefan Kangas
  2022-10-08 13:16   ` Lars Ingebrigtsen
  0 siblings, 1 reply; 3+ messages in thread
From: Stefan Kangas @ 2022-10-08 10:36 UTC (permalink / raw)
  To: Thuna, 58373

Thuna <thuna.cing@gmail.com> writes:

> Currently, `jpg' is not in image-types, despite `jpeg' being in it.
> Similarly, image descriptors of the form `(image :type jpg...)' do not
> display properly, while those of the form `(image :type jpeg...)' do.

The image type is really "JPEG" (and the MIME type "image/jpeg"), while
".jpg" just happens to be a commonly used file extension for that file
format.  So what we have now is already correct, strictly speaking.

All the same, perhaps it would be convenient to allow "jpg" as an alias
for "jpeg".  Would that risk introducing some backwards-incompatibility
with some package out there though?





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

* bug#58373: Support image-type `jpg'
  2022-10-08 10:36 ` Stefan Kangas
@ 2022-10-08 13:16   ` Lars Ingebrigtsen
  0 siblings, 0 replies; 3+ messages in thread
From: Lars Ingebrigtsen @ 2022-10-08 13:16 UTC (permalink / raw)
  To: Stefan Kangas; +Cc: 58373, Thuna

Stefan Kangas <stefankangas@gmail.com> writes:

> The image type is really "JPEG" (and the MIME type "image/jpeg"), while
> ".jpg" just happens to be a commonly used file extension for that file
> format.  So what we have now is already correct, strictly speaking.
>
> All the same, perhaps it would be convenient to allow "jpg" as an alias
> for "jpeg".  Would that risk introducing some backwards-incompatibility
> with some package out there though?

It probably won't, but I don't think it would be a good idea to
introduce an alias here.  It's less confusing to just have a simple
single mapping from type to backend.

So I don't think we should do anything here, and I'm therefore closing
this bug report.





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

end of thread, other threads:[~2022-10-08 13:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-08  1:43 bug#58373: Support image-type `jpg' Thuna
2022-10-08 10:36 ` Stefan Kangas
2022-10-08 13:16   ` Lars Ingebrigtsen

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