<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 Steve,<br><br>thanks, <br>problem occurs when one side has NONE and the other hasn't.<br>I think the test should not produce values which the normal<br>program never does (NONE with -1 for start + end).<br>Or do you find such data in Garmin maps?<br>In that case meaby it has a special meaning?<br><br>Gerd<br><br><br><div>> Date: Tue, 21 Apr 2015 10:24:47 +0100<br>> From: steve@parabola.me.uk<br>> To: mkgmap-dev@lists.mkgmap.org.uk<br>> Subject: Re: [mkgmap-dev] error in NumberPreparer?<br>> <br>> <br>> Hi Gerd<br>> <br>> > it seems that NumberPreparer (in trunk) doesn't completely ignore<br>> > the start/end values when numberStyle == NONE.<br>> <br>> I was just going to post that the patch failed with NumberRangeTest,<br>> wheras trunk passes. It seems you may have found the answer but I<br>> don't think this code should be reached when both styles are NONE<br>> since it should have exited at numbers.isEmpty() -- at least in the<br>> original code.<br>> <br>> > See line 116:<br>> > if (first.getLeftStart() > first.getLeftEnd() ||<br>> > first.getRightStart() > first.getRightEnd())<br>> > initial = Math.max(initial, rightStart);<br>> > else if (rightStart > 0)<br>> > initial = Math.min(initial, rightStart);<br>> <br>> I've got to go out now, I will try to have a look at it later.<br>> <br>> ..Steve<br>> _______________________________________________<br>> mkgmap-dev mailing list<br>> mkgmap-dev@lists.mkgmap.org.uk<br>> http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev<br></div> </div></body>
</html>