Re: [Boost-bugs] [Boost C++ Libraries] #4475: OpenVMS patch for 64 bit support

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #4475: OpenVMS patch for 64 bit support
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2011-01-13 13:09:50

#4475: OpenVMS patch for 64 bit support
----------------------------------+-----------------------------------------
  Reporter:  artyomtnk@…          |       Owner:  az_sw_dude
      Type:  Patches              |      Status:  new       
 Milestone:  Boost 1.44.0         |   Component:  date_time 
   Version:  Boost 1.44.0         |    Severity:  Problem   
Resolution:                       |    Keywords:            
----------------------------------+-----------------------------------------

Comment (by Rob Stewart):

 You propose the following structure:

 {{{
 #if defined(__VMS) && __INITIAL_POINTER_SIZE == 64
    std::tm tmp;
    if(!localtime_r(t,&tmp))
       result = 0;
    else
       *result = tmp;
 #else
    result = localtime_r(t, result);
 #endif
 }}}

 Why not this instead?  That is, why is {{{tmp}}} needed?

 {{{
 #if defined(__VMS) && __INITIAL_POINTER_SIZE == 64
    if (!localtime_r(t, result))
       result = 0;
 #else
    result = localtime_r(t, result);
 #endif
 }}}

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