Make byte-compile-cond-use-jump-table obsolete

This user option was introduced in 2017, with the rationale:

    "This is a workaround for when `byte-compile-cond-jump-table'
    accidentally generates wrong code (hasn't happened so far in my
    tests), and should be removed once we're sure there are no issues
    with it."  https://lists.gnu.org/r/emacs-devel/2017-02/msg00223.html

There hasn't been any bugs in this area within the last 6 years, and the
few we did see before that were quickly resolved.  Let's declare this
variable obsolete now, while keeping its behavior when set to nil.  We
can always revert the obsoletion if it turns out to be needed after all.

* lisp/emacs-lisp/bytecomp.el
(byte-compile-cond-use-jump-table): Declare obsolete.

(cherry picked from commit 7ba62284137ba162df16f55c75bda4d56da7dfe7)

diff --git a/lisp/emacs-lisp/bytecomp.el b/lisp/emacs-lisp/bytecomp.el
index c4d7ac0dfdd1711b5e7c4bf9e97a6d54f6ebe825..fc2c3d37e75160bad0528f9851cc70f68792678d 100644 (file)
--- a/lisp/emacs-lisp/bytecomp.el
+++ b/lisp/emacs-lisp/bytecomp.el
@@ -242,6 +242,7 @@ This includes variable references and calls to functions such as `car'."
   "Compile `cond' clauses to a jump table implementation (using a hash-table)."
   :version "26.1"
   :type 'boolean)
+(make-obsolete-variable 'byte-compile-cond-use-jump-table nil "31.1")
 
 (defvar byte-compile-dynamic nil
   "Formerly used to compile function bodies so they load lazily.