On Sat, 6 Apr 2019 at 17:18, Gilles Sadowski <gillese...@gmail.com> wrote:
>
> Le sam. 6 avr. 2019 à 16:59, Rob Tompkins <chtom...@gmail.com> a écrit :
> >
> > @Gilles - thoughts on this wording: 
> > https://github.com/apache/commons-lang/commit/3f43706192b1b75c5023f165534a12b192c31442
> >  
> > <https://github.com/apache/commons-lang/commit/3f43706192b1b75c5023f165534a12b192c31442>
> >  ?
>
> There is a typo "the user use" -> "the user uses".

Note that 'suggest that' is a subjunctive, for which 'use' is usually
considered more correct [1]

However neither reads well to me.

> Otherwise seems fair, but I'd suggest being assertive:

Agreed.

>  * "We would like to further note to users" -> "Please note that [...]"
>  * "For a more extensive treatment of random numbers" -> "For more
> stringent requirements (performance and/or correctness) [...]"
>
> That would give concrete hints that there is more to it than it looks.
>
> Thus, how about inserting this paragraph:
> ---
>  * <p>Please note that the Apache Commons project provides a component
>  * dedicated to pseudo-random number generation, namely
>  * <a href="https://commons.apache.org/rng";>Commons RNG</a>, that may be
>  * a better choice for applications with more stringent requirements
> (performance
>  * and/or correctness).</p>
> ---
> ?

That reads better.

Note: I still think we should stop short of making recommendations
regarding cryptographically secure RNGs here.
So the previous sentence "Consider instead ..." needs be dropped.

S.
[1] https://grammarist.com/grammar/subjunctive-mood/
Note: 'I suggest that he implement a ...'

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org

Reply via email to