Re: [PATCH] Extra information in autoload error

[Alex <agrambot@gmail.com>]

Eli Zaretskii <eliz@gnu.org> writes:

>> From: Alex <agrambot@gmail.com>
>> Cc: monnier@iro.umontreal.ca,  emacs-devel@gnu.org
>> Date: Sun, 23 Oct 2016 15:24:01 -0600
>> 
>> I was wondering if you preferred the version that used
>> 
>>  SDATA (Fcar (Fcar (Vload_history)))
>> 
>> or
>> 
>>  SDATA (Fcar (Fcdr (fundef)))
>> 
>> to get the extra information. The first one gets the file path, while
>> the second one just gets what was in the autoload object.
>
> Don't we want both?

That would display more information, but it would also make the error
message more verbose.

> Can you show the textual description of the
> "autoload object" that is returned by the latter form?

Here, fundef is the autoload object. That is, it's a list of the form:

  (autoload filename docstring interactive type)

So SDATA (Fcar (Fcdr (fundef))) just gets the filename of the autoload
object. The filename of the autoload object is just the 2nd argument of
the `autoload' function.

So the Vload_history version returns the whole path, while the fundef
version just uses whatever was passed into autoload. Usually that is
just the filename sans extension, e.g.:

(symbol-function 'linum-mode) => (autoload "linum" 1174895 t nil)

> (Sorry, too busy with more urgent issues to try that myself.)

No worries.