[mkgmap-dev] [Patch] Simplify UnusedElementsRemoverHook

Tue Jul 17 07:49:31 BST 2018

Hi all,

I've noticed that the hook does a few things which are probably not (or no longer) needed:
1) It checks if a way without tags is member of a relation, because the rules in relation may add tags to this way.
This was okay when the hook was added to the source, but the hook is now executed after 
those rules were executed, so that check is not needed.
2) It doesn't remove nodes outside of the tile bbox when such a node has tags keys that are used by the POI generator (--pois-to-areas-placement)
I've changed the code so that it checks for the combination of tag key and value, not only for the key.
3) The check if a way is outside of the tile bbox did not work for ways which have a bbox that intersects with the tile bbox
while the way itself neither crosses the bbox nor contains it (e.g. a large polygon).

If I hear no complains I'll commit this patch on thursday. 

Gerd
-------------- next part --------------
A non-text attachment was scrubbed...
Name: unused.patch
Type: application/octet-stream
Size: 6159 bytes
Desc: unused.patch
URL: <http://www.mkgmap.org.uk/pipermail/mkgmap-dev/attachments/20180717/6fe3db72/attachment.obj>