<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 12pt;
font-family:Calibri
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>Hi WanMil,<br><br>two points:<br>1) line 517 is obsolete:<br>mergePoints.add(end);<br>It just blows up the size of the list and processing time.<br><br>2) You were right regarding closed ways and different results.<br>The merge may create p-shaped loops.<br>See way 80605369.<br>If the northern node 381305537 is merged first:<br>2014/01/06 10:20:40 Fein (RoadMerger): e:\gerd\sp_out\63240030.o5m: Merge 33134762 and 80605369 with angle 23.159455845890676<br>and later:<br>2014/01/06 10:20:40 Fein (RoadMerger): e:\gerd\sp_out\63240030.o5m: Merge 33134762 and 80605327 with angle 44.86962367701949<br>This produced a p-shaped road (including a loop).<br><br>If the node 374244500 is processed first, no merge<br>happens because this would produce a closed loop, so<br>the node is added to mergeCompletedPoints.<br><br>The p-shaped road is split later in StyledConverter, so this is not an error,<br>but obviously it makes no sense to merge first and split again later.<br>Maybe you see a simple solution for this?<br><br>Gerd<br><br><br><br><br><br> </div></body>
</html>