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

From: Fernando Cacciola (fcacciola_at_[hidden])
Date: 2001-12-13 10:16:52

----- Original Message -----
From: Mark Rodgers <mark.rodgers_at_[hidden]>
To: <boost_at_[hidden]>
Sent: Thursday, December 13, 2001 11:42 AM
Subject: Re: [boost] Re: Formal review: New Boost.Test Library

> 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.
>
Yes, I know that.
I meant that if you add the definition of A::impl outside A, even below
foo(), it works.
I thought that because I put the definition of A::impl *below* foo(), it
would mimic the effect of having it anywhere else.

However, I realize now that since A::~A() isn't defined it shouldn't need
the definition of A::impl in this translation unit.
(which is was Gennadidy was expecting)

I've just tested it putting the definition of A::~A() and A::impl in a
different source file, and still doesn't work, as it
clearly should.

Fernando Cacciola
Sierra s.r.l.
fcacciola_at_[hidden]
www.gosierra.com