[Bro-Commits] [git/bro] topic/robin/dynamic-cast: Add experimental "is" and "as" operators. (dabe125)

Development
1

On branch : topic/robin/dynamic-cast
Link : Add experimental "is" and "as" operators. · bro/bro@dabe125 · GitHub

I've tried branching away from topic/robin/dynamic-cast for Broker
integration, but get a compile error in parse.y:

    [ 86%] [BISON][Parser] Building parser with bison 2.3
    parse.y:6.9-15: syntax error, unexpected identifier, expecting string

How do I fix it?

    Matthias

2

    [ 86%] [BISON][Parser] Building parser with bison 2.3
    parse.y:6.9-15: syntax error, unexpected identifier, expecting string

It's this line:

   %define lr.type ielr

Looks like that's a feature of newer bison versions, I'm using 3.0.4.
I added line that because that bison is otherwise having trouble with
the grammar extensions I added. I just tried 2.3 and it doesn't seem
to know that option yet. Funny thing is that if I just remove the
line, it seems to just work fine with 2.3. I haven't compiled it all
the way through but at least 2.3 isn't complaining about the
extensionss (it just needs the expected shift/reduce conflicts
adapted).

So not sure what the right solution is but for now: either upgrade
bison, or remove the line and keep an eye on if things work correctly.

Robin

3

So not sure what the right solution is but for now: either upgrade
bison, or remove the line and keep an eye on if things work correctly.

Upgrading Bison worked just fine, thanks.

    Matthias