Re: master 4a1e97b: New user option for Git log switches

[Michael Albinus]

dgutov@yandex.ru (Dmitry Gutov) writes:

Hi,

sorry for being late.

> +(defcustom vc-git-log-switches t
> +  "String or list of strings specifying switches for Git log under VC."
> +  :type '(choice (const :tag "None" t)
> +                 (string :tag "Argument String")
> +                 (repeat :tag "Argument List" :value ("") string))
> +  :version "28.1")

Is there a reason to use the t value for "None"? All other log switches
(vc-bzr-log-switches, vc-hg-log-switches, vc-src-log-switches) use nil
for this purpose, and it sounds also more natural to me as being an
empty list of strings.

Best regards, Michael.