Re: [boost] Re: Formal review: New Boost.Test Library

From: Peter Dimov (pdimov_at_[hidden])
Date: 2001-12-13 10:15:04

From: "Mark Rodgers" <mark.rodgers_at_[hidden]>
> From: "Fernando Cacciola" <fcacciola_at_[hidden]>
> > What problem exactly?
> > Giving a definition of A::Impl (which is missing from the example), it
> works
> > fine with BCB5.5.1
>
> I'm afraid that would defeat the purpose - this is an implementation of
the
> pimpl idiom, and so Impl is intended to be not defined at this point.
>
> However I'm not sure what part of the standard says this should work.  I
> guess it is implied by a few things in 12.4 and 14.7 but I'm not sure if
> the case is water-tight.

scoped_ptr<A::Impl>::~scoped_ptr should not be instantiated since it's never
referenced.

--
Peter Dimov
Multi Media Ltd.