Subject: RE: [External] : Re: bug#15809: 24.3.50; wrong defcustom type for `suggest-key-bindings'

From: Drew Adams <[email-masked]>
Date: Sat, 25 Sep 2021 15:31:46 +0000

> >> (define-widget 'natnum 'restricted-sexp
> >>   "A nonnegative integer."
> >>   :tag "Integer (positive)"
> >
> > Sorry, but this is just wrong. 0 is not a positive integer.
> 
> The text does not say that zero is positive.

The doc string doesn't, but the tag appears to.
The tag should anyway say what the field is for,
not (usually) what its type is.

If you do add indication of the type to the tag
(i.e., in addition to the field description -
what the field is for / means), then please use
what you have in the doc string.

The field value must be a nonnegative integer.
It need not be a positive integer.