Code review comment for lp:~inkscape+alexander/inkscape/clones

Revision history for this message
Mc (mc...) wrote :

Good !
I did a merge proposal to your branch to propose a simpler version of the recursive function, though it can be improved and further simplified by putting the unclip/reclip business in the unlink() function where it probably belongs.
If you find the courage, you can also propose unit tests to test the function (see testfiles/src/object-set-test.cpp )

« Back to merge proposal