Hi Gerd This change is fine by me.
Ticker On Sat, 2021-05-22 at 06:33 +0000, Gerd Petermann wrote: > Hi devs, > > I think almost nobody needs the Garmin lat/lon coordinates in log or > debug messages, the degrees are much easier to use. > This is esp. true for debugging, so I want to change Coord.toString() > to return the same result as Coord.toDegreeString(). > > The attached patch changes this and simplifies the code where > possible. > One unit test requires the garmin units, and the messages about > boundary nodes in LineClipper are probably better understandable with > garmin units. > > Unfortunately I have no easy way to find out where Coord.toString() > is called implicitly when Object.ToString() is called, so maybe there > are more places where .toGarminString() should be added. > > Any objections/hints/better ideas? > The display tool uses toDegreesString(). I think it never uses > Coord.toString() implicitly. > > Gerd > > _______________________________________________ > mkgmap-dev mailing list > mkgmap-dev@lists.mkgmap.org.uk > https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk https://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
