Re: [Boost-bugs] [Boost C++ Libraries] #10992: Chrono IO state savers inconsistent with Boost.IO state savers

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #10992: Chrono IO state savers inconsistent with Boost.IO state savers
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2015-02-16 22:59:52

#10992: Chrono IO state savers inconsistent with Boost.IO state savers
--------------------------------------+----------------------
  Reporter:  rkawulak                 |      Owner:  viboes
      Type:  Bugs                     |     Status:  assigned
 Milestone:  To Be Determined         |  Component:  chrono
   Version:  Boost Development Trunk  |   Severity:  Problem
Resolution:                           |   Keywords:
--------------------------------------+----------------------

Comment (by rkawulak):

 Yes. The second constructor should be implemented the following way to be
 consistent with IO state saver semantics (see for example
 [https://github.com/boostorg/io/blob/93932c0d55241c6d06a05b0270ca0d4a95a2f133/include/boost/io/ios_state.hpp#L119

 basic_ios_iostate_saver]):
 {{{
 duration_style_io_saver(state_type &s, aspect_type new_value) :
   s_save_(s), a_save_(get_duration_style(s))
 {
   set_duration_style(s, new_value);
 }
 }}}
 The same for all the remaining Chrono IO state savers.

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