Re: [PATCH] Extra information in autoload error

[Alex]

Eli Zaretskii <address@hidden> writes:

>> From: Alex <address@hidden>
>> Cc: address@hidden,  address@hidden
>> Date: Mon, 24 Oct 2016 13:21:08 -0600
>> 
>> 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)
>
> I think the file name alone will be enough.  Having a full absolute
> file name would be nicer, though.  I see no need for anything else:
> just the file name and the feature that was supposed to be autoloaded.
>
> Thanks.

Alright, I've attached a patch that includes both the absolute file name
and feature. If it's agreeable, then would you apply it for me?

Just to be clear, the autoload (autoload 'linum2 "linum") with this
patch will produce something like:

(error "Autoloading file /home/alex/emacs/lisp/linum.elc failed to
define function linum2")

0001-Add-extra-information-in-autoload-error.patch
Description: patch