Re: [Boost-bugs] [Boost C++ Libraries] #7718: Basic rvalue and C++11 support (part 2)

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #7718: Basic rvalue and C++11 support (part 2)
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2013-01-21 06:29:29

#7718: Basic rvalue and C++11 support (part 2)
---------------------------+------------------------------------------------
  Reporter:  apolukhin     |       Owner:  ebf         
      Type:  Patches       |      Status:  reopened    
 Milestone:  Boost 1.53.0  |   Component:  variant     
   Version:  Boost 1.52.0  |    Severity:  Optimization
Resolution:                |    Keywords:              
---------------------------+------------------------------------------------

Comment (by apolukhin):

 Now I've got the idea :-) . Yes, it is not currently implemented and will
 give a good speed boost.[[BR]]
 But in example above, won't the user be confused by v2 holding an int
 (instead of recursive_wrapper)? This proposal looks not much better than
 the first proposal, but requires more code to implement and adds 'friend'
 relationship (which is not nice). [[BR]]
 [[BR]]
 Maybe a more correct solution would be to create an additional class
 nullable_recursive_wrapper/lazy_recursive_wrapper, and possibly mark
 recursive_wrapper as a deprecated one? Or just use an
 unique_ptr/shared_ptr instead of it?

-- 
Ticket URL: <https://svn.boost.org/trac/boost/ticket/7718#comment:14>
Boost C++ Libraries <http://www.boost.org/>
Boost provides free peer-reviewed portable C++ source libraries.