Re: Fill column indicator functionality

[Basil L. Contovounesios]

Ergus <address@hidden> writes:

> branch: fill_column_indicator
> commit 780f1efe27a347c7fd92bbbc0b3bc1828749d953
> Author: Jimmy Aguilar Mena <address@hidden>
> Commit: Jimmy Aguilar Mena <address@hidden>
>
>     display-fill-column-indicator interface corrections
> ---
>  lisp/cus-start.el                     |  5 +++++
>  lisp/display-fill-column-indicator.el | 13 ++++++-------
>  2 files changed, 11 insertions(+), 7 deletions(-)
>
> diff --git a/lisp/cus-start.el b/lisp/cus-start.el
> index 3f58eac..b935776 100644
> --- a/lisp/cus-start.el
> +++ b/lisp/cus-start.el
> @@ -649,6 +649,11 @@ since it could result in memory overflow and make Emacs 
> crash."
>                                           :value nil))
>                                   "26.1")
>  
> +             (display-fill-column-indicator display-fill-column-indicator
> +                                 (choice
> +                                  (const :tag "Off (nil)" :value nil)
> +                                  (const :tag "On (t)" :value t))
> +                                  "27.1")
>               (display-fill-column-indicator-column 
> display-fill-column-indicator
>                                   integer "27.1")
>               (display-fill-column-indicator-character 
> display-fill-column-indicator

I'm not sure custom :tags should mention nil/t.  Why is :value
needed/useful?  Should this variable be defined as DEFVAR_BOOL for now,
with :type 'boolean?

Thanks,

-- 
Basil