On 19.01.2010 00:02, svn commit wrote:
Version 1498 was commited by steve on 2010-01-18 23:02:57 +0000 (Mon, 18 Jan
2010)
Reduces style rule syntax errors that are caused
by terms being in the wrong order.
Terms in the expression are now re-arranged accross the whole of the
expression and not just the first two terms. So if there is a term that can be
indexed that can be moved to the front without changing the meaning of
the expression, then it should now happen.
_______________________________________________
mkgmap-dev mailing list
[email protected]
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
rev 1498 has broken oneway handling: I put it here because the other
topic got pretty of-topic. So related to oneway=reverse handling broken:
I could lock down the problem(s).
1. with the new code, prefixing highway=* is needed for the "continue"
command to work properly (in case no highway=abc is present in the line).
2. There is no difference in how mkmgap handles oneway=-1 and
oneway=reverse. The oneway code got broken with rev: 1498 however. It is
not possible to use "set oneway=yes" on ways that already had oneway=-1
or oneway=reverse. No problem however with other oneway values such as
yes/true/1. Before my results were different, because it got broken by
1. before. So there is simply a problem with turning the direction
around of oneway=-1 and oneway=reverse and "continue".
Now we shouldn't say the "continue" code is broken, but better would be
to ditch the old conditional rules without 0x?? and replace them with
[continue with_actions]. Then oneway=-1 and oneway=reverse handling
should be put into the stylefile as a simple line:
/highway=* & ( oneway=-1 | oneway=yes ) {mkgmap:turnhighway=yes}
[continue with_actions]/
to turn around the direction of a street. The alternative would be to
turn around streets before running the style-file, and then set
oneway=-1/oneway=reverse to oneway=yes. This was the way mkgmap worked
until rev 1497.
Being able to use [continue with_actions] instead of the old conditional
code, would be much better and more comprehensible. Actually the
style-branch should behave just like this. Alternatively the behaviour
of current conditional rules should be changed to be handled as if
[continue with_actions] without assigning 0x* were used.
_______________________________________________
mkgmap-dev mailing list
[email protected]
http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev
