Hi Steve, good catch. The additional constructor is also not needed. Attached is v2 of the patch.
Gerd ________________________________________ Von: mkgmap-dev <[email protected]> im Auftrag von Steve Ratcliffe <[email protected]> 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 [email protected] http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
height_v2.patch
Description: height_v2.patch
_______________________________________________ mkgmap-dev mailing list [email protected] http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
