Re: [Boost-bugs] [Boost C++ Libraries] #2361: thread_specific_ptr: nature of the key, complexity and rationale

Subject: Re: [Boost-bugs] [Boost C++ Libraries] #2361: thread_specific_ptr: nature of the key, complexity and rationale
From: Boost C++ Libraries (noreply_at_[hidden])
Date: 2011-12-05 18:55:05

#2361: thread_specific_ptr: nature of the key, complexity and rationale
---------------------------------------+------------------------------------
  Reporter:  vicente.botet@…           |       Owner:  anthonyw           
      Type:  Feature Requests          |      Status:  new                
 Milestone:  Boost 1.37.0              |   Component:  thread             
   Version:  Boost 1.36.0              |    Severity:  Not Applicable     
Resolution:                            |    Keywords:  thread_specific_ptr
---------------------------------------+------------------------------------

Comment (by viboes):

 Replying to [comment:1 andysem]:
 > FWIW, I attached the patch against branches/release that restores
 constant-time complexity of thread_specific_ptr, instead of linear on the
 number of pointers. This essentially restores the behavior that was there
 in previous Boost versions (1.33, for sure). The patch also fixes a couple
 of other issues: calling cleanup function on NULL pointers and releasing
 cleanup function on pointer destruction.
 >
 > I ran tests on Linux (GCC 4.3) and Windows (MSVC 9) with this patch and
 it looked fine.
 >
 > Anthony, I know you're working in a different direction in trunk, thus I
 realise that this patch is unlikely to be applied to the mainline.
 However, I'd like to encourage you to provide constant-time complexity of
 TSS, like it was before Boost.Thread rewrite.

 Andrey,

 I don't know if you maintain this patch on your local system in
 synchronization with trunk. If this is the case could you attach it. I
 would like to play with to consider if your approach can be modified to
 take in account also modules support.

 Thanks

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