Re: [Boost-bugs] [Boost C++ Libraries] #7019: In libs/format/test/format_test3.cpp, 64-bit Solaris stlport4 iostream produces different result for stlport4

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #7019: In libs/format/test/format_test3.cpp, 64-bit Solaris stlport4 iostream produces different result for stlport4
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2017-10-14 12:49:23

#7019: In libs/format/test/format_test3.cpp, 64-bit Solaris stlport4 iostream
produces different result for stlport4
--------------------------------------+--------------------------------
  Reporter:  Mike Liang <mtliang@…>   |      Owner:  James E. King, III
      Type:  Bugs                     |     Status:  closed
 Milestone:  To Be Determined         |  Component:  format
   Version:  Boost Development Trunk  |   Severity:  Problem
Resolution:  wontfix                  |   Keywords:
--------------------------------------+--------------------------------
Changes (by James E. King, III):

 * status:  assigned => closed
 * resolution:   => wontfix

Comment:

 I'm going to assume this is still an issue, given that in __LP64__ mode a
 long is 8 bytes.  Here's the problem I have with the code and the patch:

 1. The format value is "x" which means integer.
 2. The value being inserted is -100 without decorations, so it should be
 treated as an integer.

 Neither of these are long, which leads me to believe the submitted patch
 is a band-aid and not fixing the root cause.  Furthermore, the boost
 project does not have a sunpro regression test in the matrix
 (http://www.boost.org/development/tests/develop/developer/format.html).  I
 don't have an environment to debug this in either.

 Further, the example you put into your description appears non-conformant
 to the standard.  The value -1 is an integer, not a long integer.  Even
 with stlsoft, _STLSOFT_SIZEOF_INT is always 4 even if _LP64 mode is on.
 (see: stlsoft-1.9.124 headers, include/stlsort/internal/cccap/sunpro.h).
 Given the failure occurs when stlsoft is added, my guess is stlsoft is to
 blame.

 I am going to close this issue as wontfix since the environment is not
 conforming to the language rules.

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