Re: (Updated) version of image-mode.el which doesn't use format-alist
- First Post
- Replies
- Stats
-
Go to
- ----- 2025 -----
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2024 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2023 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2022 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2021 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2020 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2019 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2018 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2017 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2016 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2015 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2014 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2013 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2012 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2011 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2010 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2009 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2008 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2007 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2006 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2005 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2004 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2003 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2002 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2001 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 2000 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 1999 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
- February
- January
- ----- 1998 -----
- December
- November
- October
- September
- August
- July
- June
- May
- April
- March
On Tue, 2002-10-22 at 08:52, Glynn Clements wrote:
In this version, image-mode-install removes any entries from
format-alist which have image-mode as their MODE-FN.
Patch is against the latest CVS revision (1.4).
Thanks a lot, Glynn, this one looks good to me, and it seems to work
nicely on 21.[145]. Now let's just try to get rid of those weird
"Loading image-mode..." messages on console (do you get them, BTW?), and
we're ready to replace the current image-mode with this one.
Cheers,
--
\/ille Skyttä
scop at xemacs.org
Glynn Clements
Tuesday, 22 October
Tue, 22 Oct
11:22 p.m.
New subject: (Updated) version of image-mode.el which doesn't use format-alist
Ville Skyttä wrote:
> In this version, image-mode-install removes any entries from
> format-alist which have image-mode as their MODE-FN.
>
> Patch is against the latest CVS revision (1.4).
Thanks a lot, Glynn, this one looks good to me, and it seems to work
nicely on 21.[145]. Now let's just try to get rid of those weird
"Loading image-mode..." messages on console (do you get them, BTW?),
Yes. I hadn't noticed it before because I hadn't generated the
autoloads. I've discovered that it's because of this:
;;;###autoload
(image-mode-install)
Rather than defining image-mode-install, I just inserted its body into
the file, and autoloaded that (along with image-formats-alist, which
it requires). The messages then disappear.
--
Glynn Clements <glynn.clements(a)virgin.net>
Wednesday, 23 October
Wed, 23 Oct
3:52 a.m.
New subject: (Updated) version of image-mode.el which doesn't use format-alist
>>>> "Glynn" == Glynn Clements
<glynn.clements(a)virgin.net> writes:
Glynn> I've discovered that it's because of this:
;;;###autoload
(image-mode-install)
Ah, now I remember. I guess we should look at diverting messages from
stderr to the *Warnings* buffer earlier.
Doing a *-mode-install in autoloads is generally deprecated practice.
Is it necessary/OK here?
--
Institute of Policy and Planning Sciences http://turnbull.sk.tsukuba.ac.jp
University of Tsukuba Tennodai 1-1-1 Tsukuba 305-8573 JAPAN
My nostalgia for Icon makes me forget about any of the bad things. I don't
have much nostalgia for Perl, so its faults I remember. Scott Gilbert c.l.py
Glynn Clements
Wednesday, 23 October
Wed, 23 Oct
4:29 a.m.
New subject: (Updated) version of image-mode.el which doesn't use format-alist
Stephen J. Turnbull wrote:
Glynn> I've discovered that it's because of this:
;;;###autoload
(image-mode-install)
Ah, now I remember. I guess we should look at diverting messages from
stderr to the *Warnings* buffer earlier.
Doing a *-mode-install in autoloads is generally deprecated practice.
Is it necessary/OK here?
That function originally added the auto-mode-alist entries, and was
subsequently extended to remove the format-alist entries. IOW, it
performs tasks which need to be done at startup.
In the end, I just inlined the function's body, so the actual code
goes into auto-autoloads.el rather than a call to an autoloaded
function. That seems to have solved the problem.
--
Glynn Clements <glynn.clements(a)virgin.net>
Ville Skyttä
Wednesday, 23 October
Wed, 23 Oct
7:19 p.m.
New subject: (Updated) version of image-mode.el which doesn't use format-alist
On Wed, 2002-10-23 at 06:29, Glynn Clements wrote:
In the end, I just inlined the function's body, so the actual
code
goes into auto-autoloads.el rather than a call to an autoloaded
function. That seems to have solved the problem.
Thanks for looking into this, guys. I'll do a short test drive and
commit the most recent one from -patches soon if it seems to work.
--
\/ille Skyttä
scop at xemacs.org
8413
days inactive
8414
days old
4 comments
3 participants
tags (0)
participants (3)
-
Glynn Clements -
Stephen J. Turnbull -
Ville Skyttä