> Hello WanMil
>
> Am Dienstag 18 Oktober 2011, 23:06:50 schrieb WanMil:
>
>> The patch adds a new option "pois-to-areas-placement". This option
>> defines on which node the POI generated from a polygon is placed
>> preferrably.
>>
>> Example:
>> pois-to-areas-placement=entrance=main;entrance=yes;building=entrance
>
> thanks for the new option. It looks fine but does not allow
> an empty taglist without faking trick which creates an error message for
> example via:
> --pois-to-areas-placement==
>
> So i added three lines on top of your patch that explizit allow an empty
> taglist and so disables any placements at the borderline of a polygon:
>
> /**
> * Reads the tag definitions of the option poi2area-placement-tags from
> the given properties.
> * @param props mkgmap options
> * @return the parsed tag definition list
> */
> private List<Entry<String,String>>
> getPoiPlacementTags(EnhancedProperties
> props) {
> List<Entry<String,String>> tagList = new
> ArrayList<Entry<String,String>>();
>
> String placementDefs =
> props.getProperty("pois-to-areas-placement",
> "entrance=main;entrance=yes;building=entrance");
> + placementDefs = placementDefs.trim();
> + if ( placementDefs.length() == 0 ) {
> + return tagList;
> + }
>
> String[] placementDefsParts = placementDefs.split(";");
> for (String placementDef : placementDefsParts) {
> ...
> The rest is unchanged
>
> Using this it behaves as follows(on the command line, arg file works similar):
>
> 1) no pois-to-areas-placement option given (default) is identical to:
> --pois-to-areas-placement="entrance=main;entrance=yes;building=entrance"
>
> 2) pois-to-areas-placement option given without arg always uses the center
> --pois-to-areas-placement
>
> 3) pois-to-areas-placement option given with valid arg places the node as you
> described it:
> --pois-to-areas-placement="building=entrance;entrance=*"
>
> your opinions?
> Hasemann
That's a good improvement. I will use your additional lines.
Thanks!
WanMil
_______________________________________________
mkgmap-dev mailing list
[email protected]
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
