bug#33650: 27.0.50; Root diffs between revisions

[Eli Zaretskii]

> From: Juri Linkov <juri@linkov.net>
> Cc: 33650@debbugs.gnu.org
> Date: Mon, 10 Dec 2018 02:04:30 +0200
> 
> >> It didn't work as documented before properly implemented by the patch.
> >
> > Isn't the result user-visible change in behavior?
> 
> Yes, this fix changes user-visible behavior, so perhaps it should be
> mentioned in NEWS.

I think so too.

> > I wonder whether "fileset" needs to be mentioned at all, given your
> > changes.  When does any "fileset" come into play here?
> 
> "fileset" is essential to be mentioned in `vc-version-diff'
> whereas the text "in the repository history" is unnecessary.
> 
> >> +(defun vc-root-version-diff (_files rev1 rev2)
> >> +  "Report diffs between revisions of the whole tree in the repository 
> >> history.
> >> +REV1 is an older revision, REV2 is a newer revision."
> >
> > Same here: please mention the arguments in the first sentence.
> 
> As "fileset" is essential for `vc-version-diff',
> "whole tree" is essential for `vc-root-version-diff':

OK, thanks.