Hi Mike,

thanks, commited with r3742. I am not sure about the reporting of the URLs.
I think some log messages would be very missleading when only the original id 
is reported.
Some users extract the log warning messages and publish them as a kind of todo 
list,
so I think we should make sure that the log messages contain a hint that the 
reported 
object is a result of a split or clipping or something else which modifies the 
geometry.

I did not look at the code, maybe this is already done.

Gerd


________________________________________
Von: mkgmap-dev <[email protected]> im Auftrag von Mike 
Baggaley <[email protected]>
Gesendet: Sonntag, 1. Januar 2017 23:59:47
An: 'Development list for mkgmap'
Betreff: [mkgmap-dev] patch for improvements to echotags diagnostic messages

HI Gerd, here is a patch to improve the diagnostics output by echotags when
mkgmap has created copies of data for internal processing. Currently the
original OSM Id is lost when the data is copied. This small patch retains
the original Id when elements are copied. There is no change to the map
output.

Please give this a try and commit if all is OK.

I also suggest that the toBrowseURL and getDebugName functions in
Element.java be changed to use originalId instead of id - does that seem
sensible to you?

Thanks,
Mike
_______________________________________________
mkgmap-dev mailing list
[email protected]
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev

Reply via email to