pimpl management good practice

[toddy (Todd)]

@utkarshayachit

The example you point to is not strictly PIMPL

How? The strings added to the vector are allocated on the heap and need to be freed when the vtkPythonArgsConverter instance is destroyed.

@utkarshayachit

Independently, in your example it’s not clear to me what benefit you get by allocating std::vector on stack.

It saves having to create and destroy the vector instance explicitly.