
Hi Steve,
1. Use the option set by the user. +1
2. If options is not given, look up the countries (from LBL) in the LocatorConfig; if all are drive-on-the-same-side use that.
I counted the roundabouts because they seem to be the only affected elements of this flag and I thought it might be easier to find a clean cut between different "dol" and "dor" areas. The code for that is simple enough.
3. In all other cases (and --route is given) warn loudly. OK. I'll change that in my patch.
So I'd just ignore roundabouts in the normal case. Obviously --check-roundabouts can make any checks that it likes, it is useful to debug OSM data, its not supposed to be essential to produce a map.
I never liked the idea that an option called check-xxx is also changing the data, although I am not sure if this still happens today.
Probably not. The --country-* options were added very early on since every POI has to have a country either directly or it has a region that has to have a country. At that time they were defaults for when there was no addr:country on a POI (almost always in those days!).
I suppose that if the bounds file is complete there is no need for a default, but I would expect that the --country-name should be used if there isn't one.
Okay, if I got it right we just have to add a few lines in StyledConverter. I'll post a patch for this. Gerd