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.

This archive was generated by hypermail 2.1.7 : 2017-02-16 18:50:17 UTC