hpi-swa / Ohm-S

A Squeak/Smalltalk implementation of the metaprogramming framework Ohm.
MIT License
24 stars 8 forks source link

OhmSmalltalk grammar cannot be parsed by Ohm/JS #33

Open stlutz opened 5 years ago

stlutz commented 5 years ago

I stumbled upon this using the interactive ohm editor and confirmed it by setting up ohm/JS locally.

Error: Line 207, col 7:
  206 |     | stringLiteral
> 207 |     | ByteArrayLiteral
              ^~~~~~~~~~~~~~~~
  208 |     | symbolInArrayLiteral
Cannot apply syntactic rule ByteArrayLiteral from here (inside a lexical context)

After converting ByteArrayLiteral and LiteralArrayLiteralInLiteralArray to lexical rules, it does get accepted. I'm not sure, however, if that has an impact on the grammar's correctness.

codeZeilen commented 5 years ago

Thanks for pointing it out. It does break the rule but we can change it in another way. Does this prevent progress right now?

stlutz commented 5 years ago

It shouldn't affect me, really. I decided to develop my derivative language inside Squeak, so the OhmSmalltalk grammar is correctly parsed anyway.