Re: [PATCH] Add smerge-mode, conflicted-files support to vc-git.

[Stefan Monnier <monnier@iro.umontreal.ca>]

> +(defun vc-git-find-file-hook ()
> +  "Activate `smerge-mode' if there is a conflict."
> +  (when (and buffer-file-name
> +             (vc-git-conflicted-files buffer-file-name)
> +             (save-excursion
> +               (goto-char (point-min))
> +               (re-search-forward "^<<<<<<< " nil 'noerror)))

This is the main problem, of course.  It means running git twice per
file rather than once.  We could probably improve this by only calling
vc-git-conflicted-files if the file is under Git's control and only if
it's locally modified.  If you can think of some other check we could do
to call the file even less frequently, that'd be even better.

Also, I think the check shouldn't be in find-file-hook but in
vc-git-state (so as to return `conflicted' state).

I haven't looked in detail at the patch but did notice also that
vc-git-conflicted-files is documented to take a `directory' as argument
whereas here you call it with a file name.


        Stefan