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.

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

Updates the GAT syntax #7

Merged

Conversation

arjo129
Copy link
Member

@arjo129 arjo129 commented Nov 10, 2022

As per rust-lang/rust#89122, GATs no longer use the type Typename<'a> where Self: 'a = ... syntax rather they move the where to the end.

This PR also fixes lifetimes. Seems like the type_alias_impl_trait requires you to specify a lifetime.

Signed-off-by: Arjo Chakravarty arjo@openrobotics.org

As per rust-lang/rust#89122, GATs no longer use the `type Typename<'a> where Self: 'a = ... ` syntax rather they move the where to the end.

Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
@arjo129 arjo129 linked an issue Nov 10, 2022 that may be closed by this pull request
arjo129 added a commit to open-rmf/rmf_crowdsim that referenced this pull request Nov 10, 2022
…iles with recent rust nightlies.

Mapf was not compiling with the latest nightlies causing crowdsim not to be compiling. I have updated them the branch upstream (see open-rmf/mapf#7) to make it work. Once this is merged we may close this PR.

Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
@mxgrey mxgrey merged commit de27110 into arjo/maybe_fix/visibility_timepoint Nov 17, 2022
@mxgrey mxgrey deleted the arjo/fix/update_gat_syntax branch November 17, 2022 09:17
mxgrey pushed a commit that referenced this pull request Nov 17, 2022
Signed-off-by: Arjo Chakravarty <arjo@openrobotics.org>
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.

Mapf is not compiling with latest nightlies
2 participants