Re: risky-local-variable-p called with 1 argument

[storm@cua.dk (Kim F. Storm)]

Juanma Barranquero <lektu@terra.es> writes:

> After a recent change to `risky-local-variable-p', compiling
> emacs-lisp/usafep.el gives:
> 

That change is not recorded in the lisp/ChangeLog, but I found this in
the CVS log for lisp/files.el:

revision 1.619
date: 2002/10/23 17:38:44;  author: rms;  state: Exp;  lines: +16 -11
(risky-local-variable-p): New function.
(compile-command): Use `stringp' for `safe-local-variable' prop.
(hack-one-local-variable): Use `risky-local-variable-p'.


However, that log message is mis-leading, as the change doesn't define
a new function risky-local-variable-p, but rather adds a second
mandatory VAL argument to it.

I suppose the proper log message would have been:

(risky-local-variable-p): New VAL arg.  Callers NOT changed.

And of course, the last part of that message should have rung a bell :-)



> > In unsafep-variable:
> > unsafep.el:253:6:Warning: risky-local-variable-p called with 1 argument, but
> >     requires 2
> 

-- 
Kim F. Storm <storm@cua.dk> http://www.cua.dk