Hey, I noticed that WebInputStream’s pimpl is a raw pointer.
It doesn’t need to be:
you can use a
std::unique_ptr<Pimpl> as long as your WebInputStream’s destructor is defined in the cpp file where the
Pimpl is defined.
It has to do with
unique_ptr's destructor needing to know the full definition of
Pimpl, during compilation.