logo separator

[mkgmap-dev] Don't let action rule execute its actions if its type isn't going to be used

From Mark Burton markb at ordern.com on Sun Nov 29 12:15:07 GMT 2009

Felix,

Please try this new patch to trunk. Don't use any other of my recent
style patches as it contains all the current stuff.

This now suppresses action execution when looking for further matches.
If you want the actions to be executed anyway, add 'with_actions' (or
'withactions', both are recognised).

Also, there is a subtle change in the logic in that now if an action
rule matches but otherwise would be ignored because it's not the first
time through, resolveType() will return null so that if the action rule
is part of a sequence (which it almost certainly is) then the rule
munging engine will go on to look at the next rule in the sequence.

Without that change, I think it would always choose the first rule in
cases like this:

foo=bah { tag=val1 } [0x01 continue]

foo=bah { tag=val2 } [0x02 stop]

Anyway, see what happens.

Mark
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mb-action-rule-tweak-v2.patch
Type: text/x-patch
Size: 6797 bytes
Desc: not available
Url : http://lists.mkgmap.org.uk/pipermail/mkgmap-dev/attachments/20091129/353a5aee/attachment.bin 


More information about the mkgmap-dev mailing list