$include_dir="/home/hyper-archives/boost/include"; include("$include_dir/msg-header.inc") ?>
From: David Abrahams (dave_at_[hidden])
Date: 2005-03-05 21:09:30
"Peter Dimov" <pdimov_at_[hidden]> writes:
> Jonathan Turkanis wrote:
>> David Abrahams wrote:
>>> "Peter Dimov" <pdimov_at_[hidden]> writes:
>>>
>>>>>          BOOST_BEFRIEND_TEMPLATE(shared_ptr, Y, 1)
>>>>>          BOOST_BEFRIEND_TEMPLATE(weak_ptr, Y, 1)
>>>>>          T * px;                     // contained pointer
>>>>>          detail::shared_count pn;    // reference counter
>>>>
>>>> This doesn't look any better to me than the original, sorry. ;-)
>>>
>>> I agree with Peter.  In fact, to me it looks significantly worse.
>>
>> Well, there's no accounting for taste. ;-) I use similar macros a lot
>> in the Preboost Interfaces library.
>
> This has nothing to do with taste. The original version of the code can be 
> understood immediately as it only relies on general C++ knowledge. The 
> macro-based version adds a level of indirection; you now need to look at the 
> documentation or the definition of the macros. 
Right.  It would be different if the macros significantly raised the
abstraction level, but they don't.
-- Dave Abrahams Boost Consulting www.boost-consulting.com