Commit fbc42b19 authored by Stéphane Adjemian's avatar Stéphane Adjemian

Allow th use of EXPECTATION keyword in variables tags.

parent 7fb59164
Pipeline #1196 passed with stage
in 1 minute and 57 seconds
......@@ -3899,6 +3899,8 @@ symbol : NAME
{ $$ = $1; }
| PRIOR
{ $$ = $1; }
| EXPECTATION
{ }
  • @stepan-a This is incorrect. You must put $$ = $1;, which is needed for older versions of Bison (such as the one in Debian Stretch). In more recent versions of Bison (such as the one in Debian Buster), it is no longer required, but we're not yet ready to bump the minimal Bison requirement.

    Should I fix it, or will you do it?

  • @sebastien The correct syntax for older versions does not work (this was my first attempt, but I realised that what I pushed was working as I wanted to work and didn’t realise this was a new feature). Please fix it if you can and update the submodule in ecb-master.

  • Actually I'm pretty sure it does not work the way you did it, even with a recent Bison. Did you verify that the tag really appears in the list?

    Anyways, I'm going to fix this.

  • Thinking again about it, I'm not sure it's a good idea to allow EXPECTATION here, because basically that means we can have an endogenous variable with that name. But this is also the name of an operator of the language…

    What exactly is your use case?

Please register or sign in to reply
;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment