Re: [Boost-bugs] [Boost C++ Libraries] #3339: intrusive containers vs. Sun CC

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #3339: intrusive containers vs. Sun CC
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2009-08-25 13:04:14

#3339: intrusive containers vs. Sun CC
----------------------------------------------------+-----------------------
 Reporter:  Christopher Hite <c.hite@…>             |       Owner:  igaztanaga
     Type:  Bugs                                    |      Status:  new       
Milestone:  Boost 1.40.0                            |   Component:  intrusive 
  Version:  Boost 1.39.0                            |    Severity:  Problem   
 Keywords:                                          |  
----------------------------------------------------+-----------------------

Comment(by Christopher Hite <c.hite@…>):

 Sorry, I don't think I have privileges to touch the trunk.  I only asked
 for a sandbox account.

 r55775 - I was able to fix the test code!  After removing explicit from
 smart_ptr's constructor everything works fine.  All tests pass on my
 version of Sun CC.  [[BR]]
 CC: Sun C++ 5.9 SunOS_i386 Patch 124864-12 2009/04/21

 I kind of think Sun CC is right to complain about this:
 {{{
 ".../boost/intrusive/sgtree.hpp", line 390: Error: Formal argument header
 of type
 boost::intrusive::smart_ptr<boost::intrusive::tree_node<boost::intrusive::smart_ptr<void>>>
         in call to static
 boost::intrusive::sgtree_algorithms<boost::intrusive::tree_node_traits<boost::intrusive::smart_ptr<void>>>::init_header(
 boost::intrusive::smart_ptr<boost::intrusive::tree_node<boost::intrusive::smart_ptr<void>>>)
         is being passed
 boost::intrusive::tree_node<boost::intrusive::smart_ptr<void>>*
 }}}
 You are converting node* to node_ptr implicitly in that call.  I can't
 figure out why gcc allows this.  You'd think it'd be the second "template
 <class T> smart_ptr(T *ptr)", but commenting that doesn't keep gcc from
 compiling.

 1) So please review this other tiny one line change and let me know it's
 all good.[[BR]]

 2) Please do the merging or let me know if I can.[[BR]]

 3) Someone needs to turn these tests back on for Sun 5.9.  You may need to
 upgrade it from 2007/05/03 to something this year.[[BR]]
 http://www.boost.org/development/tests/trunk/Sandia-sun.html

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