From: Niall Douglas (s_sourceforge_at_[hidden])
Date: 2022-04-06 10:01:04
On 03/04/2022 05:11, Marshall Clow via Boost wrote:
> On Apr 2, 2022, at 1:20 PM, Niall Douglas via Boost
>> Issue being fixed is https://github.com/ned14/outcome/issues/259 where
>> enabling of move assignment was not calculated correctly.
>> (To be specific, if both T and E implement move assignment and default
>> construction, but not move nor copy construction, we did not enable move
>> assignment before and with this fix now we do)
>> Code fixing it is at https://github.com/ned14/outcome/pull/260
>> Fix passes on develop branch at
> Thatâs a lot of changes, but if youâre happy with it, go ahead.
> (Looks like a lot of duplicated changes)
Turns out I forgot about `void` in the fix above.
This handles `void` correctly and I added a bunch of extra tests to
ensure it is definitely so:
Can I get permission to merge this commit to master once
says it is good?
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk