Skip to content

Conversation

@Premwoik
Copy link
Collaborator

@Premwoik Premwoik commented Feb 4, 2022

This PR addresses #28. It also fixes some abnormalities regarding token usage.

In version >= 1.13 some expressions previously without a line now have
a line propagated from a parent. This line does not always match
the location of the expression in the code. So we still need
to look at the tokens to correct these locations. This location presence
required small changes in the AstSpecifier code because now we can't be
sure when the expression has a line then this line is correct.

Changes:
- Adapt debug.ex to new elixir api
- Add specifying a line in nil.
- Propagate a generated flag from clauses to conditions.
- Consume the beginning token of list or tuple.
- Stop predicting the last line of the form from :bin or :cons expressions.
@Premwoik Premwoik force-pushed the support-elixir-v1.13 branch from e70fdd8 to 33bd901 Compare February 7, 2022 08:18
@Premwoik Premwoik marked this pull request as ready for review February 7, 2022 08:26
@Premwoik Premwoik requested a review from erszcz February 7, 2022 08:26
Copy link
Contributor

@erszcz erszcz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@erszcz erszcz merged commit a43992a into main Feb 7, 2022
@erszcz erszcz deleted the support-elixir-v1.13 branch February 7, 2022 09:20
@erszcz erszcz mentioned this pull request Feb 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants