Hi Ben,
thanks for the patch.
From my point of view it makes sense to skip an element from further
processing. I would prefer to use a different approach.
I see three possible options:
1. Implement a skip action or skip type so that the following rule skips
all POIs with no name tag created by the --add-pois-to-areas option:
name!=* & mkgmap:area2poi=true [ skip ]
or
name!=* & mkgmap:area2poi=true { skip }
I am not sure which one is the better and cleaner way. I think it is the
first one.
2. Use your delete * action but add a deleteAllTags method to the
Element class. Modify the DeleteAction class so that the deleteAllTags
method is called if the tagname is *. I think this is a cleaner way than
to put this special tag name * into the deleteTag method.
3. Implement a deleteall action that calls the Element.deleteAllTags
method described in 2.
WanMil
> Hi,
>
> I whipped up a patch to ignore unnamed POIs generated with the
> --add-pois-to-areas option. The patch adds the option to delete all
> tags with the 'delete' action using '*'. What do people think; is this
> a good approach?
>
> Thanks, Ben
>
>
>
> _______________________________________________
> 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
