diff options
author | Brecht Van Lommel <brechtvanlommel@pandora.be> | 2013-04-16 03:12:40 +0400 |
---|---|---|
committer | Brecht Van Lommel <brechtvanlommel@pandora.be> | 2013-04-16 03:12:40 +0400 |
commit | afb4b65167165613f177a531bd3d4dcb3649c1c6 (patch) | |
tree | 32e1446b5fc3ce8ec00fa0e8b9e0fcb2eedda127 /source/blender/bmesh/operators/bmo_subdivide.c | |
parent | 638b084f824bc345468bc8e02422b5da65a641a7 (diff) |
Random number generator: replace a bunch of usage of the global random number
generator with a local one. It's not thread safe and will not give repeatable
results, so in most cases it should not be used.
Also fixes #34992 where the noise texture of a displacement modifier was not
properly random in opengl animation render, because the seed got reset to a
fixed value by an unrelated function while for final render it changed each
frame.
Diffstat (limited to 'source/blender/bmesh/operators/bmo_subdivide.c')
-rw-r--r-- | source/blender/bmesh/operators/bmo_subdivide.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/source/blender/bmesh/operators/bmo_subdivide.c b/source/blender/bmesh/operators/bmo_subdivide.c index aaa2914e6ae..8df52c3a6ae 100644 --- a/source/blender/bmesh/operators/bmo_subdivide.c +++ b/source/blender/bmesh/operators/bmo_subdivide.c @@ -831,11 +831,13 @@ void bmo_subdivide_edges_exec(BMesh *bm, BMOperator *op) params.origkey = skey; if (params.use_fractal) { - BLI_srandom(seed); + RNG *rng = BLI_rng_new_srandom(seed); - params.fractal_ofs[0] = (float)BLI_drand() * 200.0f; - params.fractal_ofs[1] = (float)BLI_drand() * 200.0f; - params.fractal_ofs[2] = (float)BLI_drand() * 200.0f; + params.fractal_ofs[0] = BLI_rng_get_float(rng) * 200.0f; + params.fractal_ofs[1] = BLI_rng_get_float(rng) * 200.0f; + params.fractal_ofs[2] = BLI_rng_get_float(rng) * 200.0f; + + BLI_rng_free(rng); } BMO_slot_map_to_flag(bm, op->slots_in, "custom_patterns", |