Problem with void value type and nontrivial assignment operators

ned14 / outcome

Provides very lightweight outcome<T> and result<T> (non-Boost edition)

https://ned14.github.io/outcome

Other

675 stars 62 forks source link

Closed DiegoBaldassarMilleuno closed 2 years ago

DiegoBaldassarMilleuno commented 2 years ago

Hi! Sorry to bother you again, but it seems that there's an edge case broken in the latest fix: godbolt.org

The problem arises when T = void and E has a non-trivial copy or move assignment. Copy/move construction seems to be fine.

DiegoBaldassarMilleuno commented 2 years ago

Alright, that fixed it. Everything is passing on this end.

Props to you for fixing this so quickly! :100:

ned14 commented 2 years ago

void is the bane of all our existences!

Glad it's fixed. Just now need to get it into the upcoming Boost release. Thanks for the BR!