Patrick Dubroy

Results 27 issues of Patrick Dubroy

In the syntax documentation for productions, it says "multiple left-hand-size sentences" where (I believe) it should be "multiple right-hand-side sentences".

When we implemented incremental parsing, we [disabled two tests](https://github.com/harc/ohm/blob/2b70fa51063ee0141d4a6aa3a0537ef2900563bb/test/test-tracing.js#L374) in `test-tracing.js`: ```js test.skip('memoization', function(t) { ... }); test.skip('bindings', function(t) { ... }); ``` We should fix the underlying issues and...

chore

Back in October we switched to yarn because of workspaces. Now that workspaces are supported in npm 7, switch all references to yarn back to npm.

Here's the contents of the `npm-refs` file that contained all the places that still needed to be cleaned up: ``` CONTRIBUTING.md:17:Also, please ensure the tests pass (using `npm test`) and...

chore

It would be nice to have some built-in rules for identifiers, so that we have a sensible default for people building new languages in Ohm. For reference, I took a...

enhancement
ecmascript

Rather than a generic parse error, we should have a message like "Invalid escape sequence: \p"

Since a semantics instance is always associated with a particular, concrete grammar, we could support a shorthand that avoids calling `g.match`: ``` var g = ohm.grammar('Arithmetic { ... }'); s...

enhancement

It might be nice to have a syntax for specifying lexical binding rules for languages. Currently, the best way to do name resolution is inside an attribute/operation, but this makes...

future

``` ohm.min.js:1 Uncaught TypeError: t.toMemoKey is not a function at c.getMemoizedTraceEntry (ohm.min.js:1) at c.getTraceEntry (ohm.min.js:1) at c.eval (ohm.min.js:1) at t.Apply.o.Apply.evalOnce (ohm.min.js:1) at t.Apply.o.Apply.reallyEval (ohm.min.js:1) at t.Apply.o.Apply.eval (ohm.min.js:1) at c.eval (ohm.min.js:1)...

- stepping into a Grow LR step turns everything white - the size of the container changes as you step. - when we choose "stop into" or "step out", we...