Thread overview
[Issue 19168] std.random.randomCover and randomSample should warn that struct RNGs are copied (original is not updated)
Aug 14, 2018
Nathan S.
Aug 15, 2018
Seb
Dec 17, 2022
Iain Buclaw
August 14, 2018
https://issues.dlang.org/show_bug.cgi?id=19168

Nathan S. <n8sh.secondary@hotmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           See Also|                            |https://issues.dlang.org/sh
                   |                            |ow_bug.cgi?id=7067

--
August 15, 2018
https://issues.dlang.org/show_bug.cgi?id=19168

Seb <greensunny12@gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |greensunny12@gmail.com

--- Comment #1 from Seb <greensunny12@gmail.com> ---
Better ways:

- split in two overloads and deprecate the by value one
- fix random ranges in general by making them non-copyable (like suggested in
#7067)

However, the latter is a lot of work and it might be easier to just submit a version of mir.random as std.experimental.random and to fully replace std.random (there has been some talk about doing this, but it never materialized.)

--
December 17, 2022
https://issues.dlang.org/show_bug.cgi?id=19168

Iain Buclaw <ibuclaw@gdcproject.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P1                          |P3

--
December 01
https://issues.dlang.org/show_bug.cgi?id=19168

--- Comment #2 from dlangBugzillaToGithub <robert.schadek@posteo.de> ---
THIS ISSUE HAS BEEN MOVED TO GITHUB

https://github.com/dlang/phobos/issues/10347

DO NOT COMMENT HERE ANYMORE, NOBODY WILL SEE IT, THIS ISSUE HAS BEEN MOVED TO GITHUB

--