ShapeEditor: Always keep KnotHolders in set_item()

While looking at !377 (closed) I realized we can likely skip deleting KnotHolders altogether.

Did I miss something or is this fine?

If there are no issues and no regressions are found I'd use this to work around the very common crashing issue on 0.92.x when Ctrl/Shift-clicking on handles.