Re: [boost] [Thread] Non-copyable boost::thread

Subject: Re: [boost] [Thread] Non-copyable boost::thread
From: Anthony Williams (anthony.ajw_at_[hidden])
Date: 2010-05-17 17:52:17

"Belcourt, Kenneth" <kbelco_at_[hidden]> writes:

> On May 17, 2010, at 2:54 PM, Anthony Williams wrote:
>
>> "Belcourt, Kenneth" <kbelco_at_[hidden]> writes:
>>
>>> On May 17, 2010, at 9:49 AM, vicente.botet wrote:
>>>
>>>> From: "Belcourt, Kenneth" <kbelco_at_[hidden]>
>>>>
>>>> On May 17, 2010, at 2:25 AM, Anthony Williams wrote:
>>>>>
>>>>>> "Belcourt, Kenneth" <kbelco_at_[hidden]> writes:
>>>>>>
>>>>>>> Intel-11.1 on Darwin is a recent compiler that will not compile
>>>>>>> this code.
>>>>>>
>>>>>> Explicitly move the return value:
>>>>
>>>> If you are using Boost.Thread as I suspect, try
>>>>   return boost::move(t);
>
> With this syntax the compiler gives the inaccessible message, though  
> it's a warning not an error.
>
>>>>   return boost::thread(boost::move(t));
>
> But this works okay and is warnings free.
>
> I've attached a patch that should fix this.  Okay to commit?

No. It changes what is being tested.

> We can't use Boost libraries that have failures though expected  
> failures are okay.  That's why I'm trying to get Boost.Thread with  
> Intel on Darwin working.

test_thread_return_local is already marked up as an expected failure for
Intel/Darwin.

Anthony
-- 
Author of C++ Concurrency in Action     http://www.stdthread.co.uk/book/
just::thread C++0x thread library             http://www.stdthread.co.uk
Just Software Solutions Ltd       http://www.justsoftwaresolutions.co.uk
15 Carrallack Mews, St Just, Cornwall, TR19 7UL, UK. Company No. 5478976