<html>
  <head>
    <meta content="text/html; charset=windows-1252"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Steve,<br>
    <br>
    just testet out the intermediate mkgmap.jar.<br>
    <br>
    I can confirm that it now runs through again with both versions of
    my expression, the inital one that triggered the error as well as
    the rearranged one which I used as a workaround.<br>
    <br>
    I assume that solution will make its way into the next official
    release.<br>
    <br>
    Thanks for the quick fix and the explanations around it.<br>
    Patrik<br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 12.08.2016 15:35, Steve Ratcliffe
      wrote:<br>
    </div>
    <blockquote
      cite="mid:30488667-c60e-5ea4-3dde-d6d028c55029@parabola.me.uk"
      type="cite">
      <br>
      Hi
      <br>
      <blockquote type="cite">With newest release r3693 I'm having
        troubles creating a map with a
        <br>
        style file 'line' containing the following expression (it worked
        with
        <br>
        r3649 and earlier version):
        <br>
        <br>
            maxspeed = * & ( maxspeedkmh()>120 | maxspeed = none
        ) & ( highway =
        <br>
            motorway | highway = trunk )  { set mkgmap:road-speed-class
        = 7 }
        <br>
      </blockquote>
      <br>
      Thanks for reporting this.  I've prepared a patch attached and a
      <br>
      pre-built jar file here:
      <a class="moz-txt-link-freetext" href="http://files.mkgmap.org.uk/download/308/mkgmap.jar">http://files.mkgmap.org.uk/download/308/mkgmap.jar</a>
      <br>
      <br>
      It's possible that this fix will in turn fail on some expressions
      that
      <br>
      used to work.
      <br>
      <br>
      <blockquote type="cite">When just changing the order of the
        expressions (without changing the
        <br>
        logic/result) as shown below it works properly also with release
        r3693:
        <br>
        <br>
            maxspeed = * & ( highway = motorway | highway = trunk )
        & (
        <br>
            maxspeedkmh()>120 | maxspeed = none )  { set
        mkgmap:road-speed-class
        <br>
            = 7 }
        <br>
      </blockquote>
      <br>
      mkgmap is failing to re-arrange the expression properly, you can
      <br>
      always re-arrange it yourself by moving the more complex
      expressions
      <br>
      towards the end.
      <br>
      <br>
      Or to find bugs move them to the beginning!
      <br>
      <br>
      <blockquote type="cite">Is it possible that the 'fixes' for r3667
        or r3671 fixed some errors but
        <br>
        brought a new one in ? Or do I completely misunderstand the
        concept of
        <br>
        these expressions ?
        <br>
      </blockquote>
      <br>
      Yes, there have been a few of these reported recently.  Each bug
      that
      <br>
      is fixed has a test, so we are gradually eliminating the problems,
      <br>
      even if new ones appear.
      <br>
      <br>
      Cheers,
      <br>
      ..Steve
      <br>
      <br>
      <fieldset class="mimeAttachmentHeader"></fieldset>
      <br>
      <pre wrap="">_______________________________________________
mkgmap-dev mailing list
<a class="moz-txt-link-abbreviated" href="mailto:mkgmap-dev@lists.mkgmap.org.uk">mkgmap-dev@lists.mkgmap.org.uk</a>
<a class="moz-txt-link-freetext" href="http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev">http://www.mkgmap.org.uk/mailman/listinfo/mkgmap-dev</a></pre>
    </blockquote>
    <br>
  </body>
</html>