Skip to content

extend E0623 for earlybound and latebound for structs - #44549

Merged
bors merged 5 commits into
rust-lang:masterfrom
gaurikholkar-zz:master
Sep 17, 2017
Merged

extend E0623 for earlybound and latebound for structs#44549
bors merged 5 commits into
rust-lang:masterfrom
gaurikholkar-zz:master

Conversation

@gaurikholkar-zz

@gaurikholkar-zzgaurikholkar-zz commented Sep 13, 2017

Copy link
Copy Markdown

This fixes#44508

r? @nikomatsakis

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @nikomatsakis

(rust_highfive has picked a reviewer for you, use r? to override)

@gaurikholkar-zz

Copy link
Copy Markdown
Author

cc @arielb1

@arielb1

Copy link
Copy Markdown
Contributor

test please

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

this is still doing the wrong thing with LBRs. Please compare against the correct deBruijn index.

@nikomatsakisnikomatsakisSep 13, 2017

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

that's #44516...right?

@arielb1arielb1Sep 13, 2017

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's not fixed by #44516, and is new code.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also here

@aidanhsaidanhs added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Sep 13, 2017
@gaurikholkar-zz

Copy link
Copy Markdown
Author

@arielb1 updated

@gaurikholkar-zz

Copy link
Copy Markdown
Author

I'm guessing this is complete. Let me know if anything more is needed @nikomatsakis@arielb1

@arielb1

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Sep 14, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 9240454 has been approved by arielb1

Removing unwanted file merged in a previous commit
@gaurikholkar-zz

Copy link
Copy Markdown
Author

@arielb1 can you approve again

@arielb1

Copy link
Copy Markdown
Contributor

@bors r+

@bors

bors commented Sep 14, 2017

Copy link
Copy Markdown
Collaborator

📌 Commit 5229443 has been approved by arielb1

@alexcrichton

Copy link
Copy Markdown
Member

@bors: rollup

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Sep 16, 2017
extend E0623 for earlybound and latebound for structs
This fixesrust-lang#44508
r? @nikomatsakis
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Sep 16, 2017
extend E0623 for earlybound and latebound for structs
This fixesrust-lang#44508
r? @nikomatsakis
@frewsxcvfrewsxcv mentioned this pull request Sep 16, 2017
TimNN added a commit to TimNN/rust that referenced this pull request Sep 17, 2017
extend E0623 for earlybound and latebound for structs
This fixesrust-lang#44508
r? @nikomatsakis
@TimNNTimNN mentioned this pull request Sep 17, 2017
TimNN added a commit to TimNN/rust that referenced this pull request Sep 17, 2017
extend E0623 for earlybound and latebound for structs
This fixesrust-lang#44508
r? @nikomatsakis
@TimNNTimNN mentioned this pull request Sep 17, 2017
TimNN added a commit to TimNN/rust that referenced this pull request Sep 17, 2017
extend E0623 for earlybound and latebound for structs
This fixesrust-lang#44508
r? @nikomatsakis
@TimNNTimNN mentioned this pull request Sep 17, 2017
bors added a commit that referenced this pull request Sep 17, 2017
@bors

bors commented Sep 17, 2017

Copy link
Copy Markdown
Collaborator

☔ The latest upstream changes (presumably #44654) made this pull request unmergeable. Please resolve the merge conflicts.

@bors
bors merged commit 5229443 into rust-lang:masterSep 17, 2017
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-authorStatus: This is awaiting some action (such as code changes or more information) from the author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"these two types are declared with different lifetimes..." not triggering within structs

8 participants

@gaurikholkar-zz@rust-highfive@arielb1@bors@alexcrichton@nikomatsakis@aidanhs@gaurikholkar