[Webkit-unassigned] [Bug 98586] Extend random precision to 53bits

[bugzilla-daemon at webkit.org]

https://bugs.webkit.org/show_bug.cgi?id=98586


Oliver Hunt <oliver at apple.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #167484|review?                     |review-
               Flag|                            |




--- Comment #22 from Oliver Hunt <oliver at apple.com>  2012-11-26 11:38:49 PST ---
(From update of attachment 167484)
The more i look at this the less happy i am.  In principle I am in favour of having more random bits in the double, but the reality is that this method of getting that randomness produces a result where the low 32 bits will be extremely highly correlated with the high 21 bits.

I'd almost be tempted to pull double get() out of WeakRandom, and have a WeakRandomDouble class that has two separately initialized WeakRandom members, and use those two separate generators for the double.  This will remove the interdependency between the two.  As it is you've halved the period of the generator (something i may look at entirely independently later anyway), and you have not really gained an increase in the randomness of the low bits.

-- 
Configure bugmail: https://bugs.webkit.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.