Code review comment for lp:~inkscape.dev/inkscape/pathVectorSatellites

Revision history for this message
Krzysztof Kosinski (tweenk) wrote :

There are still a lot of issues that need to be fixed.

I am not sure about the class Pathinfo. I think these functions could be added directly to PathVector, but I'm not sure yet what Pathinfo does.

You should spend some time on writing comments what your code wants to do, it will be easier to review.

review: Needs Fixing

« Back to merge proposal