emacs-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Extra information in autoload error


From: Stefan Monnier
Subject: Re: [PATCH] Extra information in autoload error
Date: Sat, 22 Oct 2016 15:40:44 -0400
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/25.1.50 (gnu/linux)

>> if (!NILP (Fequal (fun, fundef)))
>> -    error ("Autoloading failed to define function %s",
>> +    error ("Autoloading file %s failed to define function %s",
>> +           SDATA (Fcar (Fcar (Vload_history))),

If the autoloaded file had a `require' or a `load', I suspect that
Fcar (Fcar (Vload_history)) might return the wrong file name.

> Ping. Does anyone have an opinion on which of the two diffs is better?

I'd go with the second,


        Stefan




reply via email to

[Prev in Thread] Current Thread [Next in Thread]