Re: [Emacs-diffs] master 5e43955 1/2: Don't declare vc-exec-after anymore

[Dmitry Gutov]

Hi Juanma,

On 10/22/2015 11:05 AM, Juanma Barranquero wrote:

>  > branch: master
>  > commit 5e4395544c184d9a8702cf05c7aee9329b0b9482
>  > Author: Dmitry Gutov <address@hidden <mailto:address@hidden>>
>  > Commit: Dmitry Gutov <address@hidden <mailto:address@hidden>>
>  >
>  >     Don't declare vc-exec-after anymore
...
> After this change, compiling vc-mtn.el, vc-git.el and vc-bzr.el give
> warnings (the other three don't):

Thank you for pointing that out. Any idea why that happens?

> address@hidden:\...\vc> for %i in (svn mtn hg cvs git bzr) do emacs -batch -Q
> -f batch-byte-compile vc-%i.el
>
> In end of data:
> vc-mtn.el:378:1:Warning: the function `vc-exec-after' might not be
> defined at
>      runtime.

No direct calls to vc-exec-after appear in any of these files anymore. 
All of them call vc-run-delayed, yet, as you say, the warning is 
displayed only when compiling some of them.

I'd think that if the compiler has gotten around to expanding 
vc-run-delayed, it must have already loaded vc-dispatcher. And then it 
also knows there vc-exec-after is defined.