From: Stefan Kangas <stefan@marxist.se>
Date: Sun, 26 Dec 2021 20:59:16 +0000 (+0100)
Subject: Make crm-default-separator obsolete as per FIXME
X-Git-Tag: emacs-29.0.90~1447^2~1027
X-Git-Url: http://git.eshelyaron.com/gitweb/?a=commitdiff_plain;h=693929bf48be866763edd5bc2b6e2745bec134cf;p=emacs.git

Make crm-default-separator obsolete as per FIXME

* lisp/emacs-lisp/crm.el (crm-default-separator): Make into
obsolete variable alias for crm-separator.
(crm-separator): Update docstring for above change.
---

diff --git a/lisp/emacs-lisp/crm.el b/lisp/emacs-lisp/crm.el
index 9d9c91e510e..9c49e095783 100644
--- a/lisp/emacs-lisp/crm.el
+++ b/lisp/emacs-lisp/crm.el
@@ -77,15 +77,9 @@
 
 ;;; Code:
 
-;; FIXME I don't see that this needs to exist as a separate variable.
-;; crm-separator should suffice.
-(defconst crm-default-separator "[ \t]*,[ \t]*"
-  "Default value of `crm-separator'.")
-
-(defvar crm-separator crm-default-separator
+(defvar crm-separator "[ \t]*,[ \t]*"
   "Separator regexp used for separating strings in `completing-read-multiple'.
-It should be a regexp that does not match the list of completion candidates.
-The default value is `crm-default-separator'.")
+It should be a regexp that does not match the list of completion candidates.")
 
 (defvar-keymap crm-local-completion-map
   :doc "Local keymap for minibuffer multiple input with completion.
@@ -300,6 +294,8 @@ with empty strings removed."
 ;(completing-read my-prompt my-table nil t)
 ;(completing-read my-prompt my-table nil "match")
 
+(define-obsolete-variable-alias 'crm-default-separator 'crm-separator "29.1")
+
 (provide 'crm)
 
 ;;; crm.el ends here