GNU bug report logs - #25107
Patches to clean up ispell.el

Previous Next

Package: emacs;

Reported by: Reuben Thomas <rrt <at> sc3d.org>

Date: Sun, 4 Dec 2016 17:12:02 UTC

Severity: wishlist

Tags: patch

Done: Reuben Thomas <rrt <at> sc3d.org>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: Reuben Thomas <rrt <at> sc3d.org>
To: 25107 <at> debbugs.gnu.org
Subject: bug#25107: Patches to clean up ispell.el
Date: Sun, 4 Dec 2016 17:10:42 +0000
[Message part 1 (text/plain, inline)]
The attached patch series is mostly aimed at removing XEmacs-specific code,
which is not needed, since our version of ispell.el has been part of GNU
Emacs for some time, and is not imported from upstream (the most recent
upstream release was over 20 years ago). Similarly, some workarounds for
older versions of GNU Emacs are removed (since ispell.el is not distributed
separately); in any case, most of the workarounds seem to pertain to GNU
Emacs pre-version 21.

The pre-VCS change history is also removed by the patch.

This clean-up series is aimed at making things neat before I add enchant
support (bug #17742), which will be a very simple change.

One patch also removes the pre-GNU Emacs maintainer information, so that
attentive users (who read the source!) do not mistakenly try to file bug
reports &c. with the original upstream.

I have supplied the patches as a series for ease of reading, but could
compress them if desired for installation in Emacs git.

-- 
http://rrt.sc3d.org
[Message part 2 (text/html, inline)]
[0002-Remove-ispell.el-pre-GNU-Emacs-comments.patch (text/x-patch, attachment)]
[0003-Assume-we-have-version-for-checking-ispell-version.patch (text/x-patch, attachment)]
[0004-Remove-unused-ispell-looking-back.patch (text/x-patch, attachment)]
[0005-Remove-tests-for-built-in-functions.patch (text/x-patch, attachment)]
[0006-Generalise-over-specific-documentation.patch (text/x-patch, attachment)]
[0007-Remove-some-XEmacs-specific-code-from-ispell.el.patch (text/x-patch, attachment)]
[0008-Assume-Emacs-supports-alpha-in-regexps.patch (text/x-patch, attachment)]
[0009-Remove-XEmacs-and-old-Emacs-highlighting-code.patch (text/x-patch, attachment)]
[0010-Remove-some-commented-out-code.patch (text/x-patch, attachment)]
[0011-Remove-XEmacs-specific-code-from-ispell.el.patch (text/x-patch, attachment)]
[0012-Remove-XEmacs-specific-code-dealing-with-enable-mult.patch (text/x-patch, attachment)]
[0013-Remove-XEmacs-specific-horizontal-scrollbar-handling.patch (text/x-patch, attachment)]
[0014-Remove-XEmacs-specific-minibuffer-handling-code-from.patch (text/x-patch, attachment)]
[0015-Remove-remaining-mentions-of-XEmacs-from-ispell.el.patch (text/x-patch, attachment)]

This bug report was last modified 8 years and 215 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.