Re: [Boost-bugs] [Boost C++ Libraries] #5428: No <ctime> function in Windows CE 6.0

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #5428: No <ctime> function in Windows CE 6.0
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2012-02-01 17:41:20

#5428: No <ctime> function in Windows CE 6.0
--------------------------------------------------------+-------------------
  Reporter:  Akira Takahashi <faithandbrave@…>          |       Owner:  az_sw_dude
      Type:  Bugs                                       |      Status:  new       
 Milestone:  To Be Determined                           |   Component:  date_time 
   Version:  Boost 1.46.1                               |    Severity:  Problem   
Resolution:                                             |    Keywords:  wince     
--------------------------------------------------------+-------------------

Comment (by Ulrich Eckhardt <ulrich.eckhardt@…>):

 The attached patch is step 1. It adds a time() function to c_time, which
 currently only forwards to std::time(). All uses of std::time() in
 date_time were replaced with calls to that wrapper function.

 I did two more things:
 1. In c_time.hpp were a few tabs and some foobar'd formatting, which I
 fixed along the way. There also were two misleading comments {{{#else //
 BOOST_HAS_THREADS}}} which I simply removed.
 2. In the four cases where time() was used, it was used like {{{ time_t t;
 time(&t);}}} which I replaced with a single line {{{time_t t =
 time(0);}}}. I was even wondering if I shouldn't drop the pointer argument
 completely from the wrapper, it seems of little use to me.

 I'd be happy about some feedback whether these changes are acceptable.
 I'll send the second step implementing c_time::time() for CE tomorrow, I
 still need to clean it up a bit.

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