<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'>> > Do you agree?<br><div>> ><br>> <br>> Yes. I wonder if hasEqualEndPoints() is required?<br><br>not If all routines that create shapes make sure that<br>they add the identical point to close the shape. In trunk,<br>this is not always the case, so I use it find those cases.<br>I'll post a patch later to show my results.<br><br>> <br>> But you need to take care about any way copy made within mkgmap. I think <br>> the closed flag is copied very seldom (which is wrong).<br>> <br>hmm, yes, and the docu for is_closed says<br>"+true+ the way is closed. +false+ the way is not closed and cannot be processed as polygon."<br>that would imply that we should use something like<br>way.getPoints.size() > 2 && way.hasIdenticalEndPoints()<br>to evalute is_closed() <br><br>Gerd<br></div> </div></body>
</html>