Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[futures.overview] has invalid declaration of swap for packaged_task #336

Closed
jwakely opened this issue Jun 20, 2014 · 0 comments
Closed

[futures.overview] has invalid declaration of swap for packaged_task #336

jwakely opened this issue Jun 20, 2014 · 0 comments
Assignees
@jwakely
Labels
tiny An issue with a small change; with "cwg" label: can be applied editorially after CWG consent.

Comments

@jwakely
Copy link
Member

jwakely commented Jun 20, 2014 

template <class R>
  void swap(packaged_task<R(ArgTypes...)>&,packaged_task<R(ArgTypes...)>&) noexcept;

should be

template <class R, class... ArgTypes>
  void swap(packaged_task<R(ArgTypes...)>&,packaged_task<R(ArgTypes...)>&) noexcept;

The declaration in [futures.task] and the definition in [futures.task.nonmembers] are correct.
@jwakely jwakely self-assigned this Jun 20, 2014
@jwakely jwakely added the tiny label Jun 20, 2014
@jwakely jwakely closed this as completed in 43002ce Aug 5, 2014
jwakely added a commit that referenced this issue Aug 26, 2014
@jwakely @zygoloid
[futures.overview] Add ArgTypes... to swap declaration.
d79ad59 
Fixes #336
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jwakely jwakely

Labels
tiny An issue with a small change; with "cwg" label: can be applied editorially after CWG consent.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jwakely