Óscar Fuentes <[email protected]> writes:

> Philip Jackson <[email protected]> writes:

>> P.S docstring and value are the wrong way around for
>> `magit-custom-options'.
>
> Not sure what this means.
>
> (I'm not an Elisp experienced programmer)
>

When you defined magit-custom-options you wrote


(defvar magit-custom-options
  "This variable is for internal use by the magit menu
  functionality." '())

You probably meant

(defvar magit-custom-options '()
  "This variable is for internal use by the magit menu
  functionality.")

By the way, please put meaningful docstring, it can be used by people
wanting to customize it, but also by people trying to fix bugs, or
trying to add some functionality and that could find useful to know what
the variable is used for.



-- 
Rémi Vanicat

Reply via email to