]> git.eshelyaron.com Git - emacs.git/commitdiff
Expand FIXME near definition of fboundp
authorRobert Cochran <robert-git@cochranmail.com>
Fri, 15 Jul 2016 19:45:56 +0000 (12:45 -0700)
committerNoam Postavsky <npostavs@gmail.com>
Sun, 17 Jul 2016 01:33:27 +0000 (21:33 -0400)
This expansion of the FIXME is so that future developers are aware of
the potential problems of aliasing fboundp to symbol-function without
taking backwards compatibility into account.

* src/data.c (fboundp): Note potential backwards compatibility issues in
FIXME.

Copyright-paperwork-exempt: yes

src/data.c

index 2574cbbd7645d2f128eef567fa49dfcd37a713aa..d14c84e0472195df4c80a7cb1ef61c1490afc710 100644 (file)
@@ -629,7 +629,12 @@ global value outside of any lexical scope.  */)
   return (EQ (valcontents, Qunbound) ? Qnil : Qt);
 }
 
-/* FIXME: Make it an alias for function-symbol!  */
+/* FIXME: It has been previously suggested to make this function an
+   alias for symbol-function, but upon discussion at Debbug#23957,
+   there is a risk breaking backward compatiblity, as some users of
+   fboundp may expect `t' in particular, rather than any true
+   value.  An alias is still welcome so long as the compatibility
+   issues are addressed.  */
 DEFUN ("fboundp", Ffboundp, Sfboundp, 1, 1, 0,
        doc: /* Return t if SYMBOL's function definition is not void.  */)
   (register Lisp_Object symbol)