Re: image-load-path/load-path function

[Bill Wohler <wohler@newt.com>]

Bill Wohler <wohler@newt.com> wrote:

> Reiner Steib <reinersteib+gmane@imap.cc> wrote:
> >
> > (defvar gmm-image-load-path nil
> >   "Directory where images are found.
> > See the function `gmm-image-load-path'.")
> 
> I've already renamed this to mh-image-directory per your suggestion.

While converting our code to use your version, it occurred to me that it
would be more general to use an *-image-load-path variable since the
user might have their own images in more than one place.

And then if we were to move this function and the variable to image.el,
we'd rename the variable to image-load-path. Which already exists. Which
made me think that perhaps this variable is redundant.

Since it was Peter and Debian which suggested the use of this variable
in the first place, and now that the function checks both load-path and
image-load-path for the library's images, I'd like to ask Peter if it
would be OK to change the line in emacsen-startup:

  (setq mh-image-directory "/usr/share/emacs/site-lisp/etc/images/mh-e")

to

  (if (boundp 'image-load-path)
      (add-to-list 'image-load-path
                   "/usr/share/emacs/site-lisp/etc/images/mh-e")
    (add-to-list 'load-path "/usr/share/emacs/site-lisp/etc/images/mh-e"))

Is this called late enough so that
/usr/share/emacs/site-lisp/etc/images/mh-e would be placed in front of
the standard images directory?

If this seems OK to everyone, I'll make *-image-directory a local
variable within *-image-load-path.

-- 
Bill Wohler <wohler@newt.com>  http://www.newt.com/wohler/  GnuPG ID:610BD9AD
Maintainer of comp.mail.mh FAQ and MH-E. Vote Libertarian!
If you're passed on the right, you're in the wrong lane.


-------------------------------------------------------
This SF.Net email is sponsored by xPML, a groundbreaking scripting language
that extends applications into web and mobile media. Attend the live webcast
and join the prime developer group breaking into this new coding territory!
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642