unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: Juri Linkov <juri@jurta.org>
Cc: monnier@iro.umontreal.ca, emacs-devel@gnu.org
Subject: Re: A few issues with thumbs.el
Date: Fri, 27 May 2005 17:45:05 +0300	[thread overview]
Message-ID: <87hdgon2p2.fsf@jurta.org> (raw)
In-Reply-To: <f7ccd24b05052607193e62c6b2@mail.gmail.com> (Juanma Barranquero's message of "Thu, 26 May 2005 16:19:51 +0200")

> Much, much easier to just use md5().

Even though I like when thumbnail file names are composed from file
name parts (this allows to purge old thumbnails based on their names),
this method is not reliable, because thumbnails don't get updated when
image file contents changes.

AFAIK, GIMP uses md5 for naming thumbnail files.  It stores original
file names as comments in image files.  thumb.el could do the same
adding comments with original file names, e.g.:

/usr/bin/convert -comment "Description: Thumbnail of /home/user/images/image.jpg"
                 -sample 100x100 "/home/user/images/image.jpg"
                 "png:/home/user/.emacs-thumbs/b765f25b9ed0233aaefcd601e63d6c86.png"

BTW, in thumbs.el the condition for comparing geometry is currently
commented out:

	    ;;  This is not the right fix, but I don't understand
	    ;;  the external program or why it produces a geometry
	    ;;  unequal to the one requested -- rms.
;;;	    (not (equal (thumbs-file-size tn) thumbs-geometry))

The reason why the conversion program produces a different geometry is 
because it tries to maintain the aspect ratio of the original image.

One solution is to add the geometry string to thumbnail file names, e.g.

/home/user/.emacs-thumbs/100x100_b765f25b9ed0233aaefcd601e63d6c86.png

or to add the original geometry string to the thumbnail file comments:

/usr/bin/convert -comment "Geometry: 100x100" ...

with assumption that it is possible to extract comments in Emacs
from image files before comparing them with the current value of
thumbs-geometry.

-- 
Juri Linkov
http://www.jurta.org/emacs/

  parent reply	other threads:[~2005-05-27 14:45 UTC|newest]

Thread overview: 70+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-05-25 15:18 A few issues with thumbs.el Juanma Barranquero
2005-05-25 22:24 ` Stefan Monnier
2005-05-26  0:40   ` Juanma Barranquero
2005-05-26  3:01     ` Stefan Monnier
2005-05-26  8:17       ` Juanma Barranquero
2005-05-26 14:03         ` Stefan Monnier
2005-05-26  9:37     ` Juanma Barranquero
2005-05-26 14:02       ` Stefan Monnier
2005-05-26 14:19         ` Juanma Barranquero
2005-05-26 16:55           ` Stefan Monnier
2005-05-26 18:22             ` Juanma Barranquero
2005-05-26 19:24               ` Stefan Monnier
2005-05-26 22:49                 ` Juanma Barranquero
2005-05-26 23:05                   ` Stefan Monnier
2005-05-27  7:45                     ` Juanma Barranquero
2005-05-27 14:19                     ` Richard Stallman
2005-05-27 14:45           ` Juri Linkov [this message]
2005-05-28 11:53             ` Richard Stallman
2005-05-28 18:00               ` Juanma Barranquero
2005-05-28 21:03                 ` Robert J. Chassell
2005-05-29  2:48                   ` Juanma Barranquero
2005-05-29 20:27                     ` Robert J. Chassell
2005-05-29 21:48                       ` Juanma Barranquero
2005-05-30  0:42                         ` Robert J. Chassell
2005-05-29 12:04                 ` Richard Stallman
2005-05-29 15:08                   ` Juanma Barranquero
2005-05-30 10:03             ` Juanma Barranquero
2005-05-30 18:57               ` Juri Linkov
2005-05-30 20:29                 ` Juanma Barranquero
2005-05-30 20:38                   ` David Kastrup
2005-05-30 21:05               ` Stefan Monnier
2005-05-30 21:20                 ` Juanma Barranquero
2005-05-31 17:46                 ` Richard Stallman
2005-05-31 23:20                   ` Juanma Barranquero
2005-06-09  2:00                 ` Juanma Barranquero
2005-06-09 13:22                   ` Stefan Monnier
2005-06-09 13:35                     ` Juanma Barranquero
2005-06-09 14:22                       ` Stefan Monnier
2005-06-09 14:44                         ` Juanma Barranquero
2005-06-09 15:56                           ` Stefan Monnier
2005-06-12  9:13                       ` Mathias Dahl
2005-06-12 23:40                         ` Juanma Barranquero
2005-06-12 10:45                     ` Mathias Dahl
2005-06-12 23:46                       ` Juanma Barranquero
2005-06-13  7:12                         ` Mathias Dahl
2005-06-13  7:28                           ` Juanma Barranquero
2005-06-10  0:14                   ` Richard Stallman
2005-06-10  0:38                     ` Juanma Barranquero
2005-06-10 12:37                       ` Mathias Dahl
2005-06-10 14:27                         ` Juanma Barranquero
2005-06-10 15:06                           ` Juanma Barranquero
2005-06-12  9:01                 ` Mathias Dahl
2005-06-12 10:30                   ` Henrik Enberg
2005-06-12 14:02                     ` Mathias Dahl
2005-06-12 15:33                       ` Paul Pogonyshev
2005-06-13  6:02                       ` Juri Linkov
2005-06-13  7:11                         ` Mathias Dahl
2005-06-13  7:28                         ` Mathias Dahl
2005-06-13 21:48                           ` Daniel Brockman
2005-06-14 12:53                           ` Mathias Dahl
2005-06-13  8:44                       ` Kim F. Storm
2005-06-13 14:21                         ` Mathias Dahl
2005-06-13 22:03                           ` Daniel Brockman
2005-06-13 15:03                       ` Richard Stallman
2005-06-13 16:56                         ` Paul Pogonyshev
2005-06-13 23:38                           ` Richard Stallman
2005-06-12 18:03                   ` Stefan Monnier
2005-06-13  6:01                     ` Juri Linkov
2005-06-13 15:02                     ` Richard Stallman
2005-05-26  0:48 ` Juanma Barranquero

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=87hdgon2p2.fsf@jurta.org \
    --to=juri@jurta.org \
    --cc=emacs-devel@gnu.org \
    --cc=monnier@iro.umontreal.ca \
    /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).