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 looks OK. For efficiency it first tests if the names are the same object, then null/string ordering, finally a string comparison. Ticker On Sat, 2017-07-15 at 10:49 -0700, UliBaer wrote: > 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/HousenumberGen > erator.java > Line 1537+1538: > This was already tested 6 lines above, so this cannot occur - Dead > code? > > In /mkgmap/src/uk/me/parabola/mkgmap/filters/ShapeMergeFilter.java > Line 554: > This is a String comparison, shouldn't *n1.equals(n2)* be used > instead? > > Just some thoughts of mine browsing the code - your opinions? > > > > -- > View this message in context: > http://gis.19327.n8.nabble.com/Strange-code-in-mkgmap-tp5899307.html > Sent from the Mkgmap Development mailing list archive at Nabble.com. > _______________________________________________ > mkgmap-dev mailing list > mkgmap-dev@lists.mkgmap.org.uk > http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev _______________________________________________ mkgmap-dev mailing list mkgmap-dev@lists.mkgmap.org.uk http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev