Re: [mkgmap-dev] Strange code in mkgmap?

2017-07-28 Thread Ticker Berkin
___ > Von: mkgmap-dev <mkgmap-dev-boun...@lists.mkgmap.org.uk> im Auftrag > von Ticker Berkin <rwb-mkg...@jagit.co.uk> > Gesendet: Mittwoch, 26. Juli 2017 12:12:36 > An: mkgmap-dev@lists.mkgmap.org.uk > Betreff: Re: [mkgmap-dev] Strange code in mkgmap? > > Hi Ger

Re: [mkgmap-dev] Strange code in mkgmap?

2017-07-27 Thread Gerd Petermann
et: Mittwoch, 26. Juli 2017 12:12:36 An: mkgmap-dev@lists.mkgmap.org.uk Betreff: Re: [mkgmap-dev] Strange code in mkgmap? Hi Gerd Attached is patch for ShapeSplitter Ticker On Sat, 2017-07-15 at 19:40 +0100, Ticker Berkin wrote: > The ShapeSplitter code looks like my mistake - I su

Re: [mkgmap-dev] Strange code in mkgmap?

2017-07-26 Thread Ticker Berkin
Hi Gerd Attached is patch for ShapeSplitter Ticker On Sat, 2017-07-15 at 19:40 +0100, Ticker Berkin wrote: > The ShapeSplitter code looks like my mistake - I suspect it should be > testing the lowPoint and highPoint, but need to think about this more > carefully before committing myself. > >

Re: [mkgmap-dev] Strange code in mkgmap?

2017-07-15 Thread Ticker Berkin
The ShapeSplitter code looks like my mistake - I suspect it should be testing the lowPoint and highPoint, but need to think about this more carefully before committing myself. HousenumberGenerator not mine but - yes, looks like those 2 lines should be removed ShapeMergeFilter isn't mine, but

[mkgmap-dev] Strange code in mkgmap?

2017-07-15 Thread UliBaer
Hi there, examining the code of mkgmap i found some strange lines. In /mkgmap/src/uk/me/parabola/util/ShapeSplitter.java Line 330: This is redundant? In /mkgmap/src/uk/me/parabola/mkgmap/osmstyle/housenumber/HousenumberGenerator.java Line 1537+1538: This was already tested 6 lines above, so