[PATCH] Abstract out a display-table-specific API (2)

mike.kupfer at xemacs.org mike.kupfer at xemacs.org
Sat Dec 29 14:10:30 EST 2007


>>>>> "Aidan" == Aidan Kehoe <kehoea at parhasard.net> writes:

MikeK> What I'm saying is that there should only be *one* copy in
MikeK> packages, for use by all the packages.  But your patches
MikeK> introduce duplicates in gnus, xemacs-base, misc-games, etc.
MikeK> Instead, put one copy in xemacs-base (case-table.el?  new file?),
MikeK> and all the other packages can use that copy, yeah?

Aidan> That introduces new package dependencies, something I'm not in
Aidan> a hurry to do given how much Mike Sperber fucked up the package
Aidan> build last time he did so. 

I can understand your reluctance, particularly since I don't know how
well defun-when-void and autoloads play together.  But having multiple
copies of the same function is leaving a bunch of stones around for
people to stumble over later, particularly if the function definition
ever needs to change.

I can probably be talked into adding the defuns to gnus on a temporary
basis, so that your changes to 21.5 don't get delayed too much.  But I'd
really like to have a "get well" plan first.

Would the package-future.el that Stephen recently proposed be a good
place to put put-display-table and get-display-table, using
defun-when-void?

mike



More information about the XEmacs-Beta mailing list