Re: [Boost-users] [Review] Formal Review: Boost.Move

Subject: Re: [Boost-users] [Review] Formal Review: Boost.Move
From: Steven Watanabe (watanabesj_at_[hidden])
Date: 2010-05-22 11:49:24

AMDG

Frank Mori Hess wrote:
> On Friday 21 May 2010, Neil Groves wrote:
>   
>> Sadly I'm still having errors with my version on GCC 4.4 despite having
>> fixed the aliasing problems. The relevant GCC warnings have all disappeared
>> indicating that the aliasing is now compliant.
>>     
>
> You should be aware that gcc has historically been quite poor at generating 
> strict-aliasing violation warnings.  Just because it doesn't warn doesn't 
> mean there aren't any problems.  Maybe one of those static_cast<void*>() are 
> leading to problems?

There shouldn't be a cast through void*.  Since
boost::rv<T> inherits from T, it should just use
a static_cast.  The cast back to T is effectively
a static_cast, and the two conversions need
to match.

In Christ,
Steven Watanabe