Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update grammar for order of parameters/arguments. #920

Merged
merged 2 commits into from
Jan 10, 2021
Merged

Update grammar for order of parameters/arguments. #920

merged 2 commits into from
Jan 10, 2021

Commits on Jan 6, 2021

  1. Update grammar for order of parameters/arguments. 

    Generic param order restriction was changed in rust-lang/rust#58191.
Generic argument order restriction was changed in rust-lang/rust#70261.
`for` lifetime argument restriction was changed in rust-lang/rust#48326.

Generic parameter parsing: https://github.com/rust-lang/rust/blob/206ee1eea3467fd1d7f1efdbeafe27880897bb2c/compiler/rustc_parse/src/parser/generics.rs#L83-L153
Generic argument parsing: https://github.com/rust-lang/rust/blob/17eec1433c69972844dd228b5fe801f218e118c3/compiler/rustc_parse/src/parser/path.rs#L395-L413
`for` argument parsing: https://github.com/rust-lang/rust/blob/206ee1eea3467fd1d7f1efdbeafe27880897bb2c/compiler/rustc_parse/src/parser/ty.rs#L724-L736

Fixes rust-lang#785
    @ehuss
    ehuss committed Jan 6, 2021
    Configuration menu
    Copy the full SHA
    8511cf4 View commit details
    Browse the repository at this point in the history

Commits on Jan 10, 2021

  1. Update generic params based on review.

    @ehuss
    ehuss committed Jan 10, 2021
    Configuration menu
    Copy the full SHA
    9d0aa17 View commit details
    Browse the repository at this point in the history