Re: [PATCH 1/1] zsh/random module

[Clinton Bunch <cdb_zsh@xxxxxxxxxxx>]

On 3/27/2024 18:18, Oliver Kiddle wrote:
> Clinton Bunch wrote:
> > On 3/25/2024 18:59, Oliver Kiddle wrote:
> > > Clinton Bunch wrote:
> > > > +startitem()
> > > > +vindex(SRANDOM)
> > > > +item(tt(SRANDOM)) (
> > > > +A random positive 32-bit integer between 0 and 4,294,967,295.  This parameter
> > > > +is read-only.
> > > > +)
> > > It'd be useful to spell-out exactly what the "S" in the name stands for.
> > I'm not honestly sure what it stands for, I used it for compatibility
> > with bash.  Possibly 'system'?
> Checking the bash 5.1 release notes, you may be right but it isn't clear
> from either that or the bash manual. For some reason, I was associating
> it with "secure" because the need for good random number generators is
> often associated with security.
I updated the documentation to state that the name was chosen for 
compatibility with Bash
> Before I forget this again, random should be added to the modules array
> in Completion/Zsh/Type/_module_math_func so that completion adds its
> math functions.
Done.
> Oliver