On Thu, Sep 28, 2017 at 12:59 PM John Wiegley <jwiegley@gmail.com> wrote:

> Isn't C-x v h already `vc-insert-headers'?
>

Hi John,

As mentioned earlier in the thread, Stefan suggested this binding as he
never found the default binding useful[1], Dmitry corroborated that[2], I
have never found the default useful. Later Clement and Jostein also chimed
in a +1[3, 4].

So far there has been no opposition to binding C-x v h with
vc-region-history.

I am basically proposing to bind the super-useful vc-region-history to
"something" in "C-x v" map. I offered C-x v H. But based on the above
discussion C-x v h makes more sense.

I would be great to get a clear "yes" or "no" as this issue seems to have
lingered too long.

I'd put this very concisely as:

(let ((res (binding--vc-region-history--resolution)))
  (cond ((string= "yes, C-x v h" res)
         "best option")
        ((string= "yes, C-x v H" res)
         "good option")
        ((string= "no, don't bind vc-region-history" res)
         "bad option, but I'll accept")
        (t
         (error "invalid"))))



[1]: http://lists.gnu.org/archive/html/emacs-devel/2017-09/msg00772.html
[2]: http://lists.gnu.org/archive/html/emacs-devel/2017-09/msg00786.html
[3]: http://lists.gnu.org/archive/html/emacs-devel/2017-09/msg00823.html
[4]: http://lists.gnu.org/archive/html/emacs-devel/2017-09/msg00950.html
-- 

Kaushal Modi