Doc. no.: | P0608R1 |
---|---|
Date: | 2017-12-30 |
Audience: | LEWG, LWG |
Reply-to: | Zhihao Yuan <zy at miator dot net> |
variant
constructs entirely unintended alternatives.variant<string, bool> x = "abc"; // holds bool
variant<bool, unique_ptr<int>> x = nullptr; // holds bool
The above holds string
and unique_ptr
, respectively, with the proposed fix.
variant
prefers constructions with information losses.variant<char, optional<char16_t>> x = u'\u2043'; // holds char = 'C'
double d = 3.14;
variant<int, reference_wrapper<double>> y = d; // holds int = 3
The above preserves the input value in optional<char16_t>
and reference_wrapper<double>
, respectively, with the proposed fix.
variant
performs unstable constructions.using T = variant<float, int>;
T v;
v = 0; // switches to int
T
is upgraded to variant<float, long>
,using T = variant<float, long>;
T v;
v = 0; // error
T
is upgraded to variant<float, big_int<256>>
,using T = variant<float, big_int<256>>;
T v;
v = 0; // holds 0.f
In both cases, the proposed fix consistently constructs with the second alternative.
As shown, the problems equally apply to the converting constructor and the converting assignment operator.
See also LEWG 227.
This paper proposes to constrain the variant
converting constructor and the converting assignment operator to prevent narrowing conversions and boolean conversions. This section explains what exactly this change brings.
Lemma 1. Let be a sum type . Let be a set of types that are convertible to , and be a set of types that are convertible to . Let be a set of types that are convertible to . (symmetric difference) rather than , because each causes an ambiguity.
Theorem 1. If and are extended to include a type , is shrunk.
In short, constraining variant<
>
(with ) converting constructor may enable more types to be convertible to a variant.
Definition 1. For type that is convertible to , let be a set of all the possible values for , and be a set of all the possible values for . If , is denoted as . Otherwise, and is denoted as . The conversion from to (denoted as ) is a potentially unrepresentable conversion.
In this paper, narrowing conversions (considering only the types) and boolean conversions assemble the potentially unrepresentable conversions in C++.
Lemma 2. Potentially unrepresentable conversions in C++ have Conversion rank.
The proof is left as an exercise for the reader.
Let X
be variant<
>
, r
be a value of .
When , without loss of generality, X
is variant<
>
. Effects of the proposed resolution can be summarized as follows:
X v = r; |
before | after |
---|---|---|
variant<float> v = 0; |
holds .0f |
ill-formed |
variant<float> v = INT_MAX; |
holds INT_MAX + 1 |
ill-formed |
However, variant<V>
is such a rare variant, as you can hardly say that is a sum type.
When , X
is variant<
>
.
X v = r
under the existing rule. The proposed resolution constructs as well because is not viable.X v = r
is ill-formed due to ambiguity under the existing rule. Meanwhile, it’s also ill-formed given the proposed resolution, because both conversions are potentially unrepresentable.X v = r
is still ill-formed under the existing rule. However, is constructed given the proposed resolution.X v = r
constructs under the existing rule. With the proposed resolution, it constructs instead.The effects are summarized in the order of the bullets:
X v = r; |
before | after |
---|---|---|
variant<float, vector<int>> v = 0; |
holds float |
ill-formed |
variant<float, int> v = 'a'; |
holds int('a') |
holds int('a') |
variant<float, char> v = 0; |
ill-formed | ill-formed |
variant<float, long> v = 0; |
ill-formed | holds long |
variant<float, big_int<256>> v = 0; |
holds float |
holds big_int |
When , let , be an overload set , .
X v = r
is ill-formed without the proposed resolution; orX v = r
constructs with or without the proposed resolution.The effects are summarized in the order of the bullets:
X v = r; |
before | after |
---|---|---|
variant<float, big_int<256>, big_int<128>> v = 0; |
holds float |
ill-formed |
variant<float, long, double> v = 0; |
ill-formed | holds long |
variant<float, vector<int>, big_int<256>> v = 0; |
holds float |
holds big_int |
variant<float, int, big_int<256>> v = 'a'; |
holds int |
holds int |
Theorem 2. For variant<
> v = r
, where r
is a value of , when there exists one and only one rendering to be potentially unrepresentable converted to , the proposed resolution may cause breaking changes
The first case is easy to fix while the second gives desired outcome for this paper. Both behaviors to be changed are bugs, not features, as shown in Section 1.
The author came up with and experimented a few other designs, here we list two basic ideas.
Use the alternatives’ order information in determining which one to construct. The idea defeats the purpose of the converting constructor because if the construction is sensitive to the order of the alternatives declared in the variant
template argument list, in_place_index
would be a better choice. The converting constructor and assignment operator assume unordered alternatives.
Distinguish implicit and explicit conversions. First, the idea doesn’t work well with the converting assignment operator; applying the implicit policy seems to be the only choice to maintain a consistent behavior, but this may be overkill. Second, it is counterintuitive to have an explicit constructor accepting fewer types comparing to an implicit one because of Theorem 1.
This wording is relative to N4713.
Modify 23.7.3.1 [variant.ctor]/12 as indicated:
template<class T> constexpr variant(T&& t) noexcept(see below );
Let
T
j be a type that is determined as follows: build an imaginary functionFUN
(T
i)
for each alternative typeT
i, whereFUN
(T
i)
shall not participate in overload resolution unlessT
i{t}
is well-formed and is not a boolean conversion (7.14). The overloadFUN
(T
i)
selected by overload resolution for the expressionFUN
(std::forward<T>(t)) defines the alternativeT
j which is the type of the contained value after construction.
[…]
Modify 23.7.3.3 [variant.assign]/8 as indicated:
template<class T> variant& operator=(T&& t) noexcept(see below );
Let
T
j be a type that is determined as follows: build an imaginary functionFUN
(T
i)
for each alternative typeT
i, whereFUN
(T
i)
shall not participate in overload resolution unlessT
i{t}
is well-formed and is not a boolean conversion (7.14). The overloadFUN
(T
i)
selected by overload resolution for the expressionFUN
(std::forward<T>(t)) defines the alternativeT
j which is the type of the contained value after assignment.
[…]