Created on 2023-11-15.00:00:00 last changed 3 weeks ago
Proposed resolution:
This wording is relative to N4964 after the wording changes applied by LWG 3809, which had been accepted into the working paper during the Kona 2023-11 meeting.
Modify [rand.eng.sub] as indicated:
explicit subtract_with_carry_engine(result_type value);-7- Effects: Sets the values of , in that order, as specified below. If is then , sets to ; otherwise sets to .
To set the values , first construct e, a linear_congruential_engine object, as if by the following definition:
linear_congruential_engine<uint_least32_t, 40014u,0u,2147483563u> e(value == 0u ? default_seed : static_cast<uint_least32_t>(value % 2147483563u));Then, to set each , obtain new values from successive invocations of e. Set to .
[ Wrocław 2024-11-23; Status changed: Voting → WP. ]
[ 2024-10-09; LWG telecon: Move to Ready ]
[ 2024-01-11; Reflector poll ]
Set priority to 2 after reflector poll.
More precisely, the resolution forces value
to be converted
to uint_least32_t
, which doesn't necessarily truncate, and if it
does truncate, it doesn't necessarily change the value.
But it will truncate whenever value_type
is wider than
uint_least32_t
,
e.g. for 32-bit uint_least32_t
you get a different result for
std::ranlux48_base(UINT_MAX + 1LL)()
.
The new proposed resolution below restores the old behaviour for that type.
Issue 3809 pointed out that subtract_with_carry_engine<T> can be seeded with values from a linear_congruential_engine<T, 40014u, 0u, 2147483563u> object, which results in narrowing when T is less than 32 bits. Part of the resolution was to modify the LCG seed sequence as follows:
explicit subtract_with_carry_engine(result_type value);-7- Effects: Sets the values of , in that order, as specified below. If is then , sets to ; otherwise sets to .
To set the values , first construct e, a linear_congruential_engine object, as if by the following definition:
linear_congruential_engine<result_typeuint_least32_t, 40014u,0u,2147483563u> e(value == 0u ? default_seed : value);Then, to set each , obtain new values from successive invocations of e. Set to .
Inside linear_congruential_engine, the seed is reduced modulo 2147483563, so uint_least32_t is fine from that point on. This resolution, however, forces value, the user-provided seed, to be truncated from result_type to uint_least32_t before the reduction, which generally will change the result. It also breaks the existing behavior that two seeds are equivalent if they're in the same congruence class modulo the divisor.
History | |||
---|---|---|---|
Date | User | Action | Args |
2024-11-28 21:40:31 | admin | set | messages: + msg14473 |
2024-11-28 21:40:31 | admin | set | status: voting -> wp |
2024-11-19 16:09:07 | admin | set | status: ready -> voting |
2024-10-09 16:52:10 | admin | set | messages: + msg14433 |
2024-10-09 16:52:10 | admin | set | status: new -> ready |
2024-01-11 13:02:35 | admin | set | messages: + msg13904 |
2023-11-19 11:01:08 | admin | set | messages: + msg13869 |
2023-11-15 00:00:00 | admin | create |