unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Stephen J. Turnbull" <stephen@xemacs.org>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Lars Magne Ingebrigtsen <larsi@gnus.org>,
	spinuvit@gmail.com, joakim@verona.se, emacs-devel@gnu.org
Subject: Re: imagemagic in image-mode and image-dired-thumbnail-mode?
Date: Mon, 15 Jul 2013 13:15:21 +0900	[thread overview]
Message-ID: <8761wctq5i.fsf@uwakimon.sk.tsukuba.ac.jp> (raw)
In-Reply-To: <83ehb052k1.fsf@gnu.org>

Eli Zaretskii writes:

 > How about taking out the ImageMagick code that does the scaling and
 > rolling it into Emacs?  How hard can that be?
 > 
 > Or maybe there's some other library out there that we could use to
 > roll our own image scaling code.  I don't believe for a moment that
 > ImageMagick is the only one that is able to pull that.

Eli, are you proposing to do the work yourself?  If not, ImageMagick
(or GraphicsMagick, which has the goal of being more stable, dunno if
it succeeds) are plausible strategies for getting more features for
less Emacs code, and probably less work.  (There were some other
candidates such as imlib, but I haven't followed graphics libraries
for over ten years.)

XEmacs did try the same strategy with ImageMagick and abandoned it
(too unstable, too many undocumented crashers) -- but that was over
ten years ago.  Surely ImageMagick has improved over the years.  Note
also that the decision to abandon was made by the guy who implemented
in the first place.

If the current Lisp protection from the PDF crashes isn't good enough,
add a filter at the C level so Lisp *can't* pass PDFs to ImageMagick
(or tell the ImageMagick advocates it will be removed or deprecated if
they don't, which IMO is a reasonable condition, although I probably
wouldn't require it myself).

The other issues (handling of advanced features like multi-image GIFs
<spit/>) probably aren't really bugs, but rather simply
unimplemented.  I have to wonder how important those are....



  parent reply	other threads:[~2013-07-15  4:15 UTC|newest]

Thread overview: 55+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-07-14  7:25 imagemagic in image-mode and image-dired-thumbnail-mode? Vitalie Spinu
2013-07-14  8:32 ` joakim
2013-07-14 11:48   ` Vitalie Spinu
2013-07-14 12:40     ` joakim
2013-07-14 13:01       ` Vitalie Spinu
2013-07-14 12:42     ` Lars Magne Ingebrigtsen
2013-07-14 18:21       ` Glenn Morris
2013-07-14 19:50         ` Lars Magne Ingebrigtsen
2013-07-14 20:06           ` Eli Zaretskii
2013-07-14 20:11             ` Lars Magne Ingebrigtsen
2013-07-14 22:00             ` Vitalie Spinu
2013-07-15  4:38               ` Eli Zaretskii
2013-07-15  4:15             ` Stephen J. Turnbull [this message]
2013-07-15  4:46               ` Eli Zaretskii
2013-07-15  5:45                 ` Stephen J. Turnbull
2013-07-15 10:39                   ` Óscar Fuentes
2013-08-02 15:32                     ` Steinar Bang
2013-07-15 15:50                   ` Eli Zaretskii
2013-07-14 18:33   ` Glenn Morris
2013-07-14 19:17     ` joakim
2013-07-15 10:51       ` Vitalie Spinu
2013-07-16 15:57         ` Glenn Morris
2013-07-16 21:26         ` Stefan Monnier
2013-07-17  7:29           ` Vitalie Spinu
2013-07-17 15:51             ` Vitalie Spinu
2013-07-18  8:47               ` Lars Magne Ingebrigtsen
2013-07-18 22:27                 ` Vitalie Spinu
2013-07-19  9:22                   ` Stefan Monnier
2013-07-20  7:25                     ` Vitalie Spinu
2013-07-22 20:17                 ` Vitalie Spinu
2013-07-22 20:31                   ` Lars Magne Ingebrigtsen
2013-07-23  8:31                     ` Vitalie Spinu
2013-07-18 23:22               ` image-transform.el and image-mode.el rewrite Vitalie Spinu
2013-07-19 11:52                 ` Wolfgang Jenkner
2013-07-19 12:21                   ` Wolfgang Jenkner
2013-07-20  7:18                     ` Vitalie Spinu
2013-07-22 20:37                 ` Glenn Morris
2013-07-22 21:05                   ` Vitalie Spinu
2013-10-08 18:08                   ` Glenn Morris
2013-10-08 23:43                     ` Vitalie Spinu
2013-10-09  0:02                       ` Michael Heerdegen
2014-12-15  9:33                         ` Vitalie Spinu
2014-12-18 14:17                           ` Michael Heerdegen
2014-12-18 21:32                             ` Vitalie Spinu
2014-12-18 15:15                           ` Stefan Monnier
2014-12-18 23:23                             ` Vitalie Spinu
2014-12-19  4:19                               ` Stefan Monnier
2014-12-19  4:46                                 ` Vitalie Spinu
2014-12-19  8:56                                 ` Eli Zaretskii
2014-12-19 17:50                                   ` Stefan Monnier
2014-12-19 19:37                                     ` Eli Zaretskii
2014-12-19 21:31                                       ` Stefan Monnier
2014-12-19 21:49                                         ` Eli Zaretskii
2014-12-19 10:24                                 ` Vitalie Spinu
2014-12-19 17:51                                   ` Stefan Monnier

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/emacs/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8761wctq5i.fsf@uwakimon.sk.tsukuba.ac.jp \
    --to=stephen@xemacs.org \
    --cc=eliz@gnu.org \
    --cc=emacs-devel@gnu.org \
    --cc=joakim@verona.se \
    --cc=larsi@gnus.org \
    --cc=spinuvit@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).