APPROVE COMMIT sjt-xft
Courtesy notice of private branch commit.
>>>> "Clemens" == Clemens Fruhwirth
<clemens(a)endorphin.org> writes:
Clemens> Obviously the else path has not been removed when the if
Clemens> statement for font-running-xemacs was removed.
Clemens> The "patch" below removes the remains.
Thanks, I'll apply that, with this ChangeLog:
2005-09-07 Clemens Fruhwirth <clemens(a)endorphin.org>
* font.el (font-default-font-for-device): Remove vestigial code
from incomplete removal of conditional.
Clemens> Stephen: The patch from Daniel Pittman
Clemens>
http://list-archive.xemacs.org/xemacs-patches/200507/msg00058.html
Clemens> gives a nice speed gain. He hasn't merged the patch
Clemens> himself yet. Would you like to do that, while you're at
Clemens> the code? :)
Actually, that's been in for weeks, along with some refactoring of
that series of functions which may have confused the issue. Please
check and make sure that you do have the right file after a CVS
update. You should find that the only call to an Fc*Init* () function
is in the XEmacs init code at the end of the file, and the only call
to FcFontList now has a NULL FcConfig argument. If not, let me know
and we'll debug our CVS (cvs diff against the file I have claims that
that's what is in CVS).
--
School of Systems and Information Engineering
http://turnbull.sk.tsukuba.ac.jp
University of Tsukuba Tennodai 1-1-1 Tsukuba 305-8573 JAPAN
Ask not how you can "do" free software business;
ask what your business can "do for" free software.