<html><head><meta http-equiv="Content-Type" content="text/html; charset=UTF-8" /></head><body style='font-size: 10pt; font-family: Verdana,Geneva,sans-serif'>
<p>I am not sure how far mkgmap should exceed its original scope and become a QA tool... But maybe an option to configure the output format of the height and conv functions (w.r.t. decimal separator, thousands separator, number of digits etc) might be useful? Perhaps the program could take (some of) the settings from the user's current locale.</p>
<div> </div>
<p>--colin</p>
<p>On 2017-02-03 11:37, Gerd Petermann wrote:</p>
<blockquote type="cite" style="padding: 0 0.4em; border-left: #1010ff 2px solid; margin: 0"><!-- html ignored --><!-- head ignored --><!-- meta ignored -->
<div class="pre" style="margin: 0; padding: 0; font-family: monospace">Hi all,<br /> <br /> it is quite easy to implement the support for comma as optional decimal separator, but<br /> much more difficult to produce a reasonable warning. The problem is that the routine<br /> which detects the error may be called several times when style rules are evaluated.<br /> Attached is a patch which creates a warning for each evaluation.<br /> With the default style the messages look like this:<br /> WARN: uk.me.parabola.mkgmap.osmstyle.actions.ConvertFilter f:\osm\peaks.osm: invalid decimal separator found in value '13,3' <a href="http://www.openstreetmap.org/node/-31655">http://www.openstreetmap.org/node/-31655</a><br /> WARN: uk.me.parabola.mkgmap.osmstyle.actions.ConvertFilter f:\osm\peaks.osm: invalid decimal separator found in value '13,3' <a href="http://www.openstreetmap.org/node/-31655">http://www.openstreetmap.org/node/-31655</a><br /> WARN: uk.me.parabola.mkgmap.osmstyle.actions.ConvertFilter f:\osm\peaks.osm: invalid decimal separator found in value '13,3m' <a href="http://www.openstreetmap.org/node/-31660">http://www.openstreetmap.org/node/-31660</a><br /> WARN: uk.me.parabola.mkgmap.osmstyle.actions.ConvertFilter f:\osm\peaks.osm: invalid decimal separator found in value '13,3m' <a href="http://www.openstreetmap.org/node/-31660">http://www.openstreetmap.org/node/-31660</a><br /> <br /> The corresponding OSM file is attached, a binary with the patch is here:<br /> <a href="http://files.mkgmap.org.uk/download/332/mkgmap.jar">http://files.mkgmap.org.uk/download/332/mkgmap.jar</a><br /> <br /> @Steve: Please check the change in the regex pattern, I think the dot in the existing pattern should be quoted ?<br /> <br /> Gerd<br /> <br /> <br /> ________________________________________<br /> Von: mkgmap-dev <<a href="mailto:mkgmap-dev-bounces@lists.mkgmap.org.uk">mkgmap-dev-bounces@lists.mkgmap.org.uk</a>> im Auftrag von Andrzej Popowski <<a href="mailto:popej@poczta.onet.pl">popej@poczta.onet.pl</a>><br /> Gesendet: Donnerstag, 2. Februar 2017 16:38:37<br /> An: <a href="mailto:mkgmap-dev@lists.mkgmap.org.uk">mkgmap-dev@lists.mkgmap.org.uk</a><br /> Betreff: Re: [mkgmap-dev] Is height: filter working as described?<br /> <br /> Hi Carlos,<br /> <br /> maybe mkgmap could issue a warning for filter whenever input data are<br /> not compatible? Comma is a standard decimal separator in Poland and<br /> there is a lot of erroneous tags. I haven't noticed this problem until now.<br /> <br /> When numeric value contains a comma, filter "height" doesn't convert tag<br /> value but adds a a separator. The result is that Mapsource doesn't<br /> display label at all.<br /> <br /> --<br /> Best regards<br /> <br /> _______________________________________________<br /> mkgmap-dev mailing list<br /> <a href="mailto:mkgmap-dev@lists.mkgmap.org.uk">mkgmap-dev@lists.mkgmap.org.uk</a><br /> <a href="http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev">http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev</a></div>
<br />
<div class="pre" style="margin: 0; padding: 0; font-family: monospace">_______________________________________________<br /> mkgmap-dev mailing list<br /> <a href="mailto:mkgmap-dev@lists.mkgmap.org.uk">mkgmap-dev@lists.mkgmap.org.uk</a><br /> <a href="http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev">http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev</a></div>
</blockquote>
</body></html>