[josm-dev] [PATCH] Some core changes to support validator plugin

Dave Hansen dave at sr71.net
Wed Nov 21 15:31:26 GMT 2007


On Wed, 2007-11-21 at 14:35 +0100, Gabriel Ebner wrote:
> So if I understand you correctly, this patch doesn't actually fix the
> problem?

I think the real problem is that I get orphan nodes without this patch.
That's an actual data inconsistency, and I bet there are worse ones that
pop up than just that.  It would be really trivial to loop on the fix
button as long as progress is being made.  We can actually measure that
now with success/fail coming back from executeAction().

Perhaps we need some more descriptive actions for these kinds of things.
Instead of ChangeAction() we should have WayNodeReplaceAction(way,
oldnode, newnode) or something similar.  

-- Dave





More information about the josm-dev mailing list