Re: Bug#141446: gettext-el: emacs has error loading 50gettext.el (fwd)

[Miles Bader]

Daniel Quinlan <address@hidden> writes:
...
> (if (and (boundp 'emacs-version)
>        (not (string-lessp emacs-version "20"))
>        (not (string-match "XEmacs" emacs-version)))
>     (modify-coding-system-alist 'file "\\.po[tx]?\\'\\|\\.po\\."
>                               'po-find-file-coding-system))

Assuming that _if_ xemacs has a `modify-coding-system-alist' function,
then it's compatible, then it would be more robust (and concise) to just
test for that:

   (if (fboundp 'modify-coding-system-alist)
       (modify-coding-system-alist 'file "\\.po[tx]?\\'\\|\\.po\\."
                                   'po-find-file-coding-system))

In general, it's a bad idea to test version numbers unless there's no
alternative.

-Miles
-- 
Next to fried food, the South has suffered most from oratory.
                        -- Walter Hines Page