Opened 11 years ago

Closed 10 years ago

#6194 closed Feature Requests (fixed)

Adapt to Boost.Move

Reported by: viboes Owned by: viboes
Milestone: Boost 1.50.0 Component: thread
Version: Boost 1.48.0 Severity: Problem
Keywords: move semantics Cc: viboes

Description

As suggested in the ML Boost.Thread should move to Boost.Move to implement move semantics emulation.

Attachments (2)

6194_0.patch (42.7 KB ) - added by Evan Wallace <onlyone@…> 11 years ago.
Version 0 of the patch to #6194
6194_1.patch (37.5 KB ) - added by Evan Wallace <onlyone@…> 11 years ago.
Version 1 of the patch to #6194

Download all attachments as: .zip

Change History (11)

comment:1 by viboes, 11 years ago

Cc: viboes added

comment:2 by viboes, 11 years ago

See also:

#2380 boost::move from lvalue does not work with gcc

#2501 upgrade_to_unique_lock doesn't compile on C++0x mode

#4521 Error using boost::move on packaged_task (MSVC 10)

#4705 C++03 Move Support Undocumented

#4711 Must use implementation details to return move-only types

#6141 Compilation error when boost.thread and boost.move are used together

#6174 [thread] packaged_task does correctly handle moving results

These tickets could be closed once the emulation is adopted and of course the problem disappears.

This is one is related but will need more modifications.

#5990 shared_future<T>::get() has wrong return type

comment:3 by viboes, 11 years ago

See also #6222 Compile error with SunStudio: unique_future move

by Evan Wallace <onlyone@…>, 11 years ago

Attachment: 6194_0.patch added

Version 0 of the patch to #6194

comment:4 by Evan Wallace <onlyone@…>, 11 years ago

Sorry about the long delay.

I have attached a patch should fix this, as well as #2380, #2501, #4521, #6141, #6174 and #5990.

The patch is relative to 1.48.0. The patch does not incorporate any changes made to the trunk since 1.48.0, and also does not incorporate the fixes to #6174 and #5990 that were attached to those tickets by viboes (I had already fixed the problems independently, before noticing about the existence of those patches).

The attached patch is missing tests and documentation. I will send a longer description and justification of the changes to the mailing list.

by Evan Wallace <onlyone@…>, 11 years ago

Attachment: 6194_1.patch added

Version 1 of the patch to #6194

comment:5 by Evan Wallace <onlyone@…>, 11 years ago

The new patch should address most of the concerns raised in the mailing list so far. The patch is again relative to 1.48.0, and it is still missing tests and documentation.

comment:6 by viboes, 11 years ago

Owner: changed from Anthony Williams to viboes
Status: newassigned

comment:7 by viboes, 11 years ago

Milestone: To Be DeterminedBoost 1.49.0

Committed in trunk at revision [76543].

comment:8 by viboes, 11 years ago

Severity: ShowstopperProblem

comment:9 by viboes, 10 years ago

Milestone: Boost 1.49.0Boost 1.50.0
Resolution: fixed
Status: assignedclosed

Committed in release branch at [78543]

Note: See TracTickets for help on using tickets.