> On 9 Oct 2024, at 16:05, Heikki Linnakangas <hlinn...@iki.fi> wrote:

Thanks for looking!

> I guess the '1000' was supposed to be the maximum, but ParseVariableDouble 
> doesn't take a maximum.

Doh, I had a max parameter during hacking but removed it since I didn't see a
clear usecase for it.  Since it's not an externally published API we can
alwasys add it when there is need.  Clearly I managed to generate the patch at
the wrong time without noticing.  Fixed.

> That's a little inaccurate: 0 is also accepted, so should be "must be greater 
> than *or equal to* 0". Or maybe "cannot be negative". -0 is also accepted, 
> though.

I changed to "must be at least XX" to keep the message short.

> set -> sets
> 
> beetween -> between

Fixed.

--
Daniel Gustafsson

Attachment: v2-0001-Make-default-watch-interval-configurable.patch
Description: Binary data

Reply via email to