[mkgmap-dev] Is height: filter working as described?

Fri Feb 3 09:58:41 GMT 2017

Hi Steve,

good catch. The additional constructor is also not needed. Attached is v2 of the patch.

Gerd
________________________________________
Von: mkgmap-dev <mkgmap-dev-bounces at lists.mkgmap.org.uk> im Auftrag von Steve Ratcliffe <steve at parabola.me.uk>
Gesendet: Freitag, 3. Februar 2017 10:53:20
An: Development list for mkgmap
Betreff: Re: [mkgmap-dev] Is height: filter working as described?

On 02/03/2017 07:32 AM, Gerd Petermann wrote:
> I'd prefer not to change the syntax because that would break backward compatibility.
>
> Attached is a small patch which allows to use height without an argument. If an arg is given it must use
> ft or feet as target. The doc is changed accordingly.
> I've also tried to fix a problem in the docu for conv.

+       public HeightFilter(String s) {
+               super(s == null ? "m=>s" : s);

There is a typo there I think, should be "m=>ft"

..Steve
_______________________________________________
mkgmap-dev mailing list
mkgmap-dev at lists.mkgmap.org.uk
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
-------------- next part --------------
A non-text attachment was scrubbed...
Name: height_v2.patch
Type: application/x-download
Size: 2063 bytes
Desc: height_v2.patch
URL: <http://www.mkgmap.org.uk/pipermail/mkgmap-dev/attachments/20170203/234f218b/attachment-0001.bin>