Expose dbRecursiveShapeIterator::next(RecursiveShapeReceiver*)

[rocallahan]

This is extremely useful when you want the flexibility of pull-based iteration, but you also need notifications when entering and leaving cells.

[klayoutmatthias]

Could you explain the context? Are you working on a C++ extension? Is there a project link?

This is not exposing the function externally in a way. The patch makes sense if you have C++ code based on this class. But in general I discourage that. There is no warranty for the C++ API. It may change without notice.

Matthias

[rocallahan]

Could you explain the context? Are you working on a C++ extension? Is there a project link?

We have some C++ code using KLayout as a library. That code is not open-source (and is only used internally).

This is not exposing the function externally in a way. The patch makes sense if you have C++ code based on this class. But in general I discourage that. There is no warranty for the C++ API. It may change without notice.

OK, that makes sense. Thanks for the warning. We'll deal with whatever changes you make.

I will assume you don't want this kind of change, then. We'll carry it locally. Thanks!!!

[klayoutmatthias]

Well, thanks for this clarification. I don't object to the change, but as there is no specific use within KLayout, I might as well refactor away this function or change its functionality. So accepting this PR would not mean anything, except a potentially short-living feature.