frenetic-lang / frenetic

The Frenetic Programming Language and Runtime System
http://www.frenetic-lang.org/
Other
222 stars 52 forks source link

Avoid illegal _= bindings in Parser.cppo.mly. #625

Closed fpottier closed 5 years ago

fpottier commented 5 years ago

Hello,

Using a wildcard pattern "_" in a place where a variable name is expected has never been legal, even though (by luck) it was accepted until now by Menhir. It will likely be disallowed in the next release, so I suggest avoiding these illegal bindings.

The existing opam package descriptions for frenetic should also be updated to require Menhir <= 20181026.

Cheers, François.

smolkaj commented 5 years ago

Thanks for the patch!