Skip to content

Commit ce756f7

Browse files
committed
remove constexpr qualifiers for certain Result member functions
1 parent 7a86847 commit ce756f7

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

include/common/result.hpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -100,7 +100,7 @@ class Result {
100100
template<typename U, typename E>
101101
requires std::constructible_from<T, U>
102102
&& (std::same_as<std::remove_cvref_t<E>, Errs> || ...)
103-
constexpr Result(U&& value, E&& error)
103+
Result(U&& value, E&& error)
104104
: value(std::forward<U>(value)),
105105
error(std::forward<E>(error)) {}
106106

@@ -112,7 +112,7 @@ class Result {
112112
*/
113113
template<typename E>
114114
requires Sentinel<T> && (std::same_as<std::remove_cvref_t<E>, Errs> || ...)
115-
constexpr Result(E&& error)
115+
Result(E&& error)
116116
: error(std::forward<E>(error)),
117117
value(sentinel_v<T>) {}
118118

@@ -123,7 +123,7 @@ class Result {
123123
*/
124124
template<typename E>
125125
requires(std::same_as<E, Errs> || ...)
126-
constexpr std::optional<E> get() const& {
126+
std::optional<E> get() const& {
127127
if (std::holds_alternative<E>(error)) {
128128
return std::get<E>(error);
129129
} else {
@@ -138,7 +138,7 @@ class Result {
138138
*/
139139
template<typename E>
140140
requires(std::same_as<E, Errs> || ...)
141-
constexpr std::optional<E> get() && {
141+
std::optional<E> get() && {
142142
if (std::holds_alternative<E>(error)) {
143143
return std::move(std::get<E>(error));
144144
} else {

0 commit comments

Comments
 (0)