Re: [Boost-bugs] [Boost C++ Libraries] #6126: Signed integer members of Boost.Fusion adapted ADTs are not output correctly with Boost.Spirit.Karma rules

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #6126: Signed integer members of Boost.Fusion adapted ADTs are not output correctly with Boost.Spirit.Karma rules
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2012-01-10 17:15:51

#6126: Signed integer members of Boost.Fusion adapted ADTs are not output
correctly with Boost.Spirit.Karma rules
-------------------------------+--------------------------------------------
  Reporter:  t0rt1e@…          |       Owner:  hkaiser                                                                                                             
      Type:  Bugs              |      Status:  new                                                                                                                 
 Milestone:  To Be Determined  |   Component:  spirit                                                                                                              
   Version:  Boost 1.48.0      |    Severity:  Regression                                                                                                          
Resolution:                    |    Keywords:  patch proposed, Boost.Spirit.Karma, Boost.Fusion, BOOST_FUSION_ADAPT_CLASS, BOOST_FUSION_ADAPT_ADT, short, int, long
-------------------------------+--------------------------------------------
Changes (by t0rt1e@…):

  * keywords:  Boost.Spirit.Karma, Boost.Fusion, BOOST_FUSION_ADAPT_CLASS,
               BOOST_FUSION_ADAPT_ADT, short, int, long =>
               patch proposed, Boost.Spirit.Karma,
               Boost.Fusion, BOOST_FUSION_ADAPT_CLASS,
               BOOST_FUSION_ADAPT_ADT, short, int, long
  * severity:  Problem => Regression

Comment:

 I have tested Jeroen Habraken's and Seth Heren's patches against
 {{{test_signed_integer_output_with_karma_minimal.cpp}}}. Like suggested by
 Seth, copying the {{{attr}}} value in {{{any_int_generator<T,
 CharEncoding, Tag, Radix, force_sign>::insert_int< OutputIterator,
 Attribute>(sink, attr)}}} has reliably resolved the problem for me on Mac
 OS X Lion 10.7.2 when compiling with g++ 4.2.1, 4.5.3, 4.6.2 and clang++
 3.0 version. I kept the structure of the {{{return}}} statement and just
 replace {{{attr}}} with {{{attr_copy}}}.

 I attach a patch against Boost svn rev. 76398.

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