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 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

Reply via email to