Boost logo

Boost :

Subject: Re: [boost] [random] new threefry random engine
From: Beman Dawes (bdawes_at_[hidden])
Date: 2014-04-20 12:23:44


On Sun, Apr 20, 2014 at 11:18 AM, Mathias Gaunard <
mathias.gaunard_at_[hidden]> wrote:

> On 18/04/14 10:00, Thijs van den Berg wrote:
>
>
>>> Do not use reinterpret_cast in operator().
>>> The behavior is undefined, and will certainly
>>> differ depending on endianness.
>>>
>> I did this for speed and I didn’t worry about endianness because it never
>> crossed my mind. Changing the order of bits/bytes should not affect the
>> quality of the randomness, but it’s indeed not nice to have different
>> behaviour on different machines. I’ll fix this.
>>
>
> Personally I would want this engine to produce exactly the same results as
> the reference random123 implementation for a given seed.
>

+1

> This should be the case regardless of the platform used.
>

+1

--Beman


Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk