Skip to content

Commit

Permalink
Update 3003 discussion to refer to 2245
Browse files Browse the repository at this point in the history
  • Loading branch information
jwakely committed Sep 20, 2024
1 parent 5e81bf7 commit 7440029
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions xml/issue3003.xml
Original file line number Diff line number Diff line change
Expand Up @@ -101,13 +101,16 @@ That was removed by issue <iref ref="2921"/>, but issue <iref ref="2976"/>
argues that there was no good reason to do that. Removing `uses_allocator`
for `packaged_task` would have made sense (as proposed below for `promise`)
but 2921 didn't do that (which is why 2976 was needed).
If we restore that `packaged_task` constructor then we need to fix `reset()`
as discussed in issue <iref ref="2245"/>.
</p>
<p>In summary:
<ul>
<li>Keep support for using an allocator for shared state of a `promise`.</li>
<li>Remove `uses_allocator` specialization for `promise`.</li>
<li>Restore support for using an allocator for the shared state of `packaged_task`.</li>
<li>Do not restore `uses_allocator` specialization for `packaged_task`.</li>
<li>Update `packaged_task::reset()` to deal with allocators.</li>
</ul>
</p>

Expand Down

0 comments on commit 7440029

Please sign in to comment.