tree-sitter-go icon indicating copy to clipboard operation
tree-sitter-go copied to clipboard

Published 20 hours ago • verified publisher icon tree-sitter

Materialize expression_statement nodes

Open adonovan opened this issue 3 years ago • 2 comments

This change causes the parser to materialize expression_statement nodes for calls and receive expressions that appear as a statement, so that every child of a block is a statement (or comment).

REVIEWERS: why does test --update make a bunch of unnecessary whitespace changes to the corpus/*.txt files? I'm using ./node_modules/tree-sitter-cli/tree-sitter (v0.19.5) for this step. Should I be doing something different?

Checklist:

  • [ ] All tests pass in CI.
  • [ ] There are sufficient tests for the new fix/feature.
  • [ ] Grammar rules have not been renamed unless absolutely necessary.
  • [ ] The conflicts section hasn't grown too much.
  • [ ] The parser size hasn't grown too much (check the value of STATE_COUNT in src/parser.c).

adonovan avatar Mar 28 '22 18:03 adonovan

every child of a block is a statement

This seems like a good change.

why does test --update make a bunch of unnecessary whitespace changes?

It is reformatting the S-expressions. I'm ok with committing these changes. For now, we'll just need to verify that no ERROR or MISSING nodes have been introduced.

maxbrunsfeld avatar Mar 28 '22 21:03 maxbrunsfeld

Can you upgrade to the latest Tree-sitter and regenerate with that? Currently, the latest version is 0.20.6.

maxbrunsfeld avatar Mar 28 '22 21:03 maxbrunsfeld

Ping. We'd like to have materialized expression statements in cursorless: https://github.com/cursorless-dev/cursorless/issues/1592.

I'd weakly prefer to only include call and receive expressions (and set aside the spec restrictions around certain predeclared identifiers), but what is in this PR is definitely good enough for us for now.

(Also: hi, @adonovan! Just about every time I have a Go-related issue, I find you've already got a fix ready for it. Amazing.)

josharian avatar Jul 10 '23 18:07 josharian

Could you rebase?

amaanq avatar Jul 10 '23 18:07 amaanq

I know this is Alan's PR, but in case it is helpful, I rebased this commit and regenerated with tree-sitter 0.20.8. (Please do double-check my work if you want to use it.)

https://github.com/josharian/tree-sitter-go/commit/309e76588c33af4b9b48ea2b37751be01b21e253

Note that I found that regenerating at master with 0.20.8 made changes as well, so those would be mixed in here.

josharian avatar Jul 10 '23 19:07 josharian

If you want you can PR that, but remove the comments added, especially the LL one - but otherwise I think it looks good

amaanq avatar Jul 10 '23 19:07 amaanq

Sent https://github.com/tree-sitter/tree-sitter-go/pull/124.

josharian avatar Jul 10 '23 23:07 josharian

Superseded by #124

amaanq avatar Jul 11 '23 23:07 amaanq