Possible improvement for MultiBlock deep copies

Hi all,

Today, I was questioning myself on the deep copy mechanism of the vtkMultiBlockDataSet vs. the shallow copy one.

In the Shallow Copy, there is preservation of the metadata even if the child block’s data object is nullptr

It looks like it is not the case for Deep Copy here: VTK/vtkDataObjectTree.cxx at 732d680d307699d4c6a1a10ae17f7202b300cd6d · Kitware/VTK · GitHub

In other words, block metadata does not get transferred when the data object is nullptr.

I believe there are reasons behind this, but if there aren’t any, can I issue an MR that fixes this?

Thanks

Looks like an oversight, however we may change mbds::ShallowCopy soon, so lets hold unto this for a few days.

@spyridon97