Re: [boost] Review Request: impl_ptr (pimpl)

Subject: Re: [boost] Review Request: impl_ptr (pimpl)
From: Vladimir Batov (Vladimir.Batov_at_[hidden])
Date: 2017-08-15 22:00:41

On 08/15/2017 09:57 PM, Seth via Boost wrote:
> On 15-08-17 02:51, Vladimir Batov via Boost wrote:
>> That might happen to be a deal breaker but that's the best I could
>> come up with to get it to work. If anyone has a better working
>> solution, I am all ears.
> I have the same interest. I'd love to know what the global declaration
> is "helping" work. I have the intuition that it is not necessary in any
> way if you properly account for ADL.
>
> Can you point at an explanation of _what purpose_ is served, how, by
> having ::impl_ptr in the global namespace? I'm happy to toy with it and
> possibly create a PR.

The thread called "Pimpl Again?" dated June 2016 on this list might be a 
good start. In short, the reason is the specialization of 
impl_ptr<Foo>::implementation.

> In the light of that, it would help if there were tests.

impl_ptr/test is the test directory -- 
https://github.com/yet-another-user/pimpl/tree/master/test.