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

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() 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