[josm-dev] Jumbo Patch

Gabriel Ebner ge at gabrielebner.at
Mon Dec 17 23:33:36 GMT 2007


On Mon, Dec 17, 2007 at 03:15:01PM -0800, Dave Hansen wrote:
> > - Sometimes we actually want to change all occurencies of a node (and not just
> >   the first one).  ReplaceAllNodesInWay?  (DuplicateNode,
> >   MotorwayIntersections, DeleteAction)
> 
> I wonder if we should be doing it like that, or if we should be more
> careful about detecting the duplicate nodes up front.  

When fixing a duplicated node, we delete the superfluous nodes.  If we don't
replace all occurencies in a way, we leave behind references to deleted nodes,
thus rendering the data set inconsistent.

Same when deleting a node.  If you only replace one occurence, then the other
will point to a deleted node.

  Gabriel.




More information about the josm-dev mailing list