all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: npostavs@users.sourceforge.net
To: Xue Fuqiao <xfq@gnu.org>
Cc: William Stevenson <yhvh2000@gmail.com>,
	10772@debbugs.gnu.org, Stefan Monnier <monnier@iro.umontreal.ca>
Subject: bug#10772: 24.0.93; [patch] Convert tcl-auto-fill-mode to use define-minor-mode
Date: Fri, 02 Jun 2017 23:45:18 -0400	[thread overview]
Message-ID: <87fufh20q9.fsf@users.sourceforge.net> (raw)
In-Reply-To: <87ppm9qyze.fsf@gnu.org> (Xue Fuqiao's message of "Thu, 27 Feb 2014 06:40:37 +0800")

[-- Attachment #1: Type: text/plain, Size: 246 bytes --]

Xue Fuqiao <xfq@gnu.org> writes:

> +(define-obsolete-function-alias tcl-auto-fill-mode
> +  auto-fill-mode "24.4")

I think making this an alias breaks backwards compatibility needlessly.
I propose just marking it obsolete without changing it:


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: patch --]
[-- Type: text/x-diff, Size: 1814 bytes --]

From dd1ba59961cc216422349716fcfc62e656076500 Mon Sep 17 00:00:00 2001
From: Noam Postavsky <npostavs@gmail.com>
Date: Sat, 1 Apr 2017 21:02:50 -0400
Subject: [PATCH v1] Make tcl-auto-fill-mode obsolete (Bug#10772)

* lisp/progmodes/tcl.el (tcl-auto-fill-mode): Declare obsolete.
* etc/NEWS: Announce it.
---
 etc/NEWS              | 5 +++++
 lisp/progmodes/tcl.el | 5 +++--
 2 files changed, 8 insertions(+), 2 deletions(-)

diff --git a/etc/NEWS b/etc/NEWS
index 7972511f7a..cbd388b216 100644
--- a/etc/NEWS
+++ b/etc/NEWS
@@ -1306,6 +1306,11 @@ window changed size when 'window-size-change-functions' are run.
 *** The semantics of 'mouse-autoselect-window' has changed slightly.
 For details see the section "Mouse Window Auto-selection" in the Elisp
 manual.
+
+---
+** 'tcl-auto-fill-mode' is now declared obsolete.  It's functionality
+can be replicated simply by setting 'comment-auto-fill-only-comments'.
+
 \f
 * Changes in Emacs 26.1 on Non-Free Operating Systems
 
diff --git a/lisp/progmodes/tcl.el b/lisp/progmodes/tcl.el
index 902a5aace0..de0cd50911 100644
--- a/lisp/progmodes/tcl.el
+++ b/lisp/progmodes/tcl.el
@@ -353,8 +353,6 @@ (defvar tcl-mode-hook nil
     Quotes all \"#\" characters that don't correspond to actual
     Tcl comments.  (Useful when editing code not originally created
     with this mode).
-  `tcl-auto-fill-mode'
-    Auto-filling of Tcl comments.
 
 Add functions to the hook with `add-hook':
 
@@ -1413,6 +1411,9 @@ (defun tcl-restart-with-file (file &optional and-go)
 
 (defun tcl-auto-fill-mode (&optional arg)
   "Like `auto-fill-mode', but sets `comment-auto-fill-only-comments'."
+  (declare
+   (obsolete
+    "Use `auto-fill-mode' with `comment-auto-fill-only-comments'." "26.1"))
   (interactive "P")
   (auto-fill-mode arg)
   (if auto-fill-function
-- 
2.11.1


  parent reply	other threads:[~2017-06-03  3:45 UTC|newest]

Thread overview: 16+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-02-09  6:49 bug#10772: 24.0.93; [patch] Convert tcl-auto-fill-mode to use define-minor-mode William Stevenson
2012-02-09 15:18 ` Juanma Barranquero
2012-02-09 22:09 ` Stefan Monnier
2012-02-11 15:27   ` William Stevenson
2012-02-12  5:12     ` Stefan Monnier
2014-02-25 13:30       ` Xue Fuqiao
2014-02-25 14:30         ` Stefan Monnier
2014-02-25 22:25           ` Xue Fuqiao
2014-02-26 15:40             ` Stefan Monnier
2014-02-26 22:40               ` Xue Fuqiao
2014-02-27  2:55                 ` Stefan Monnier
2014-02-27 17:00                 ` Glenn Morris
2014-02-27 22:33                   ` Xue Fuqiao
2016-02-24  6:42                     ` Lars Ingebrigtsen
2017-06-03  3:45                 ` npostavs [this message]
2017-06-28  0:38                   ` npostavs

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=87fufh20q9.fsf@users.sourceforge.net \
    --to=npostavs@users.sourceforge.net \
    --cc=10772@debbugs.gnu.org \
    --cc=monnier@iro.umontreal.ca \
    --cc=xfq@gnu.org \
    --cc=yhvh2000@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/emacs.git
	https://git.savannah.gnu.org/cgit/emacs/org-mode.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.