result

result copied to clipboard

Fix for issue 17: rename `concept` to `concepts` to fix clash with C++20 keyword

3

#17

willowell

yisonPylkita

`namespace concept` is invalid in C++20-enabled compilers because `concept` is then a keyword.

willowell

See https://stackoverflow.com/questions/1238016/are-compound-statements-blocks-surrounded-by-parens-expressions-in-ansi-c for reasoning.

hellow554

https://github.com/oktal/result/blob/fee9af7e0c775f0148810f9449fa8354b3eec569/result.h#L862-L873 Shouldn't a simple `return false;` in the end fix it?

hellow554

Could you add an example for mapError function usage?

gpbicego

``` template class #if __cplusplus >= 201703L [[nodiscard]] #endif Result { }; ``` This is c++17 feature and will show warning if you will ignore the Result

Yanpas

Is it still possible to use andThen/bind ? "then" is not what i'm looking for (i think).

TestResultType

Move value on storage construction

1

Required for non-copy constructible types.

vorot93

Right now this library contains two versions of `Result`: one with normal `Ok` enum and another with `void` one. It may be a short-term convenience for callers (simple Ok() suffices...

vorot93