procyberian/emacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Expand FIXME near definition of fboundp

Browse source 
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
This commit is contained in:
Robert Cochran 2016-07-15 12:45:56 -07:00 • committed by Noam Postavsky
parent 219b39f521
commit 9356fe2392
1 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/data.c
View file

@ -629,7 +629,12 @@ global value outside of any lexical scope. */)
return (EQ (valcontents, Qunbound) ? Qnil : Qt); 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, DEFUN ("fboundp", Ffboundp, Sfboundp, 1, 1, 0,
doc: /* Return t if SYMBOL's function definition is not void. */) doc: /* Return t if SYMBOL's function definition is not void. */)
(register Lisp_Object symbol) (register Lisp_Object symbol)