Subject: [boost] [outcome] Changelog of all changes made since review announcement one week ago
From: Niall Douglas (s_sourceforge_at_[hidden])
Date: 2017-05-19 00:54:48
The Outcome review begins later today, but so much feedback has landed
this past week already that I had better announce what has changed in
develop branch to prevent people reraising already fixed things:
## 20170519 boost_peer_review4:
- Issue #13 The opcode counting CI tests in test/constexprs was severely
broken on POSIX. Thanks to Tom Westerhout for finding the problem and
supplying its fix.
- unittests.cpp had become clang-tidy unclean, fixed.
- Issue #15 Move `outcome::expected<T, E>` into
`outcome::experimental::expected<T, E>`. Thanks to Vicente J. Botet
Escriba for persuading me to do this.
- Issue #14 Removed all the static asserts in the Expected
implementation apart from the nothrow move constructor check, which is
now correct unlike the incorrect assert from before. Thanks to Vicente
J. Botet Escriba for explaining Expected's semantics.
- Issue #21 Correct misspelling of `extended_error_code` with
`error_code_extended` in the docs. Thanks to Deniz Bahadir for reporting
- Issue #17 Give credit to Paul Bristow for the name "Outcome". Thanks
to Peter Dimov for finding the correct person.
- Issue #18 `result<T>` had a `.set_exception()` which it should not
have had. Thanks to Peter Dimov for reporting this.
- Issue #19 Remove observers `.is_ready()`, `.get_error_or()`,
`.get_exception_or()`. Thanks to Peter Dimov and Andrzej Krzemienski for
- Issue #20 Rename:
- `monad_error` to `bad_outcome`
- `monad_errc` to `bad_outcome_errc`
- `monad_category` to `bad_outcome_category`
- `BOOST_OUTCOME_THROW_MONAD_ERROR` to `BOOST_OUTCOME_THROW_BAD_OUTCOME`
Thanks to Peter Dimov, Andrzej Krzemienski and Paul Bristow for
If anyone prefers to see these changes in a tarball, you can get a
or the boost_peer_review4 tag in git. Note that this tarball was
generated straight off develop branch from my laptop and MAY NOT WORK well.
I didn't get a chance to implement these outstanding issues before the
- https://github.com/ned14/boost.outcome/issues/11 Expected should never
have a valueless state
- https://github.com/ned14/boost.outcome/issues/12 Copy assignment gets
instantiated even when type T cannot be copied
- https://github.com/ned14/boost.outcome/issues/16 Default constructor
of T is sometimes compiled when T has no default constructor
Thanks in advance for your feedback and reviews. As I hope the above
shows, it makes a big difference.
-- ned Productions Limited Consulting http://www.nedproductions.biz/ http://ie.linkedin.com/in/nialldouglas/
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk