Re: [Boost-bugs] [Boost C++ Libraries] #3470: Bug in boost::date_time::c_time::localtime(), boost::date_time::c_time::gmtime()

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #3470: Bug in boost::date_time::c_time::localtime(), boost::date_time::c_time::gmtime()
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2009-09-21 14:53:38

#3470: Bug in boost::date_time::c_time::localtime(),
boost::date_time::c_time::gmtime()
-----------------------------------------------+----------------------------
 Reporter:  Farid Zaripov <faridz@…>           |       Owner:  andysem  
     Type:  Bugs                               |      Status:  new      
Milestone:  Boost 1.41.0                       |   Component:  date_time
  Version:  Boost 1.40.0                       |    Severity:  Problem  
 Keywords:  gmtime, localtime                  |  
-----------------------------------------------+----------------------------
Changes (by andysem):

  * severity:  Showstopper => Problem

Comment:

 Hmm... According to the docs the functions should return the pointer to tm
 passed as an argument, so technically this is a bug. However, I don't see
 a good rationale for this requirement as it only introduces unnecessary
 data copying. Also, this code is there for ages and no one has brought it
 up. Maybe we're better off correcting the docs. And a new, more C++-style
 interface might also be a good idea.

 Do you have a rationale for this change? A motivating example, perhaps?

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