Re: New function: vc-ediff

[Christoph Scholtes]

Stefan Monnier <address@hidden> writes:

>> address@hidden vc-ediff
>> +The function @code{vc-ediff} works similar to @code{vc-diff} and
>> +provides a way to visually compare two revisions of a file an Ediff
>                                                             ^^^
>                                                             in?

Yes. Fixed.

> No need to give any detail about a new function's purpose, just "New
> function" is enough: the source code should explain the purpose (either
> self-evidently or via comments&docstrings).
> Also, remove the empty lines to make it clear that those three file-changes
> go together.

OK. Fixed also.

> Problem with this code is that you depend on ediff-revision to get
> the revision arguments.  E.g. this doesn't provide completion.
> IOW we should use more of vc-diff's code here.

It was my intention to reuse the code that ediff-revision already
provides to get the revision arguments. I am not sure I understand what
you mean by "does not provide completion".

Do you mean vc-diff's code to get the revision arguments?

Christoph