|
Boost : |
From: David Abrahams (dave_at_[hidden])
Date: 2003-01-30 11:28:03
"Peter Dimov" <pdimov_at_[hidden]> writes:
>> I'm not all set up to run those tests and measure the times, which is
>> why I was hoping Philippe would check it out.
>
> There is a test in libs/smart_ptr/test called
> shared_ptr_alloc_test.cpp that you can use.
OK.
>> > quick_allocator doesn't compile on VC6 or Borland 5.5.1, though. I
> haven't
>> > investigated too deeply.
>>
>> You must have changed the code I sent then. Works fine with VC6 for
>> me; borland has the usual problems with ICEs and scope qualification.
>
> I did.
OK.
>> This was just a quick hack; I'm surprised you thought it was worthwhile
>> modifying your sources to accomodate it despite the lack of *any* test
>> data...
>
> The modification to shared_count.hpp is small and macro-guarded, so it won't
> affect existing code. Seeing how well a "quick hack" performs can be
> valuable information, given how often the "optimized count allocator"
> question comes up.
Sure; I was just surprised you wanted to check it in this early.
>> or did you try it?
>
> This is almost an insult.
Unintentional. Please don't be insulted; I guess I'm just afraid I'll
be blamed for adding "the little optimization that didn't" ;-)
-- David Abrahams dave_at_[hidden] * http://www.boost-consulting.com Boost support, enhancements, training, and commercial distribution
Boost list run by bdawes at acm.org, gregod at cs.rpi.edu, cpdaniel at pacbell.net, john at johnmaddock.co.uk