Skip to content
This repository has been archived by the owner on Dec 18, 2019. It is now read-only.

feat(parser) Implement class-constant-access-expression #116

Merged
merged 9 commits into from
Sep 13, 2017

Conversation

Hywan
Copy link
Member

@Hywan Hywan commented Sep 12, 2017

Address #68.

Try to implement the `class-constant-access-expression`. To avoid an
infinite type size, it is required to change the definition of
`DereferencableExpression` a little bit by adding `Box` to `Expression`
and `Array` variants.
It tries to implement the `class-constant-access-expression` rule.
`class_constant_access` can create a conflict with other parsers like
`literal` or `array`. To avoid stopping at an `Incomplete` and try
another branch of the `alt!` macro, it is replaced by `alt_complete!`.

`class_constant_access` is also moved to the last branches of
`alt_complete!` to avoid many backtracks.
@coveralls
Copy link

coveralls commented Sep 12, 2017

Coverage Status

Coverage decreased (-0.05%) to 95.63% when pulling f22dba1 on Hywan:class-constant-access into 338bc27 on tagua-vm:master.

1 similar comment
@coveralls
Copy link

coveralls commented Sep 12, 2017

Coverage Status

Coverage decreased (-0.05%) to 95.63% when pulling f22dba1 on Hywan:class-constant-access into 338bc27 on tagua-vm:master.

@coveralls
Copy link

coveralls commented Sep 12, 2017

Coverage Status

Coverage decreased (-0.05%) to 95.63% when pulling f22dba1 on Hywan:class-constant-access into 338bc27 on tagua-vm:master.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.05%) to 95.63% when pulling f22dba1 on Hywan:class-constant-access into 338bc27 on tagua-vm:master.

@Hywan Hywan merged commit f22dba1 into tagua-vm:master Sep 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants