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: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Reuben Thomas <rrt <at> sc3d.org>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#25107: closed (Patches to clean up ispell.el)
Date: Tue, 13 Dec 2016 19:47:01 +0000
[Message part 1 (text/plain, inline)]
Your message dated Tue, 13 Dec 2016 19:45:55 +0000
with message-id <CAOnWdohGNskTXhyF9vC=aNJOfdfL_w6Dr+AgOzrE_O1Pt61dUA <at> mail.gmail.com>
and subject line 
has caused the debbugs.gnu.org bug report #25107,
regarding Patches to clean up ispell.el
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
25107: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=25107
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Reuben Thomas <rrt <at> sc3d.org>
To: bug-emacs <bug-emacs <at> gnu.org>
Subject: Patches to clean up ispell.el
Date: Sun, 4 Dec 2016 17:10:42 +0000
[Message part 3 (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 4 (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)]
[Message part 19 (message/rfc822, inline)]
From: Reuben Thomas <rrt <at> sc3d.org>
To: 25107-done <at> debbugs.gnu.org
Date: Tue, 13 Dec 2016 19:45:55 +0000
[Message part 20 (text/plain, inline)]

[Message part 21 (text/html, inline)]

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

Previous Next


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