Skip to content

save-analysis: get tables directly, accomodating them being missing #39285

New issue

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

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

Already on GitHub? # to your account

Merged
merged 1 commit into from
Jan 28, 2017

Conversation

nrc
Copy link
Member

@nrc nrc commented Jan 25, 2017

Fixes an ICE when running with save-analysis after an error

r? @eddyb

@eddyb
Copy link
Member

eddyb commented Jan 25, 2017

@bors r+

@bors
Copy link
Collaborator

bors commented Jan 25, 2017

📌 Commit 6d2fc9c has been approved by eddyb

@bors
Copy link
Collaborator

bors commented Jan 25, 2017

⌛ Testing commit 6d2fc9c with merge 3f5da5b...

@bors
Copy link
Collaborator

bors commented Jan 25, 2017

💔 Test failed - status-travis

@alexcrichton
Copy link
Member

alexcrichton commented Jan 25, 2017 via email

@bors
Copy link
Collaborator

bors commented Jan 26, 2017

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

Fixes an ICE when running with save-analsysis after an error
@nrc
Copy link
Member Author

nrc commented Jan 26, 2017

@bors: r=@eddyb

@bors
Copy link
Collaborator

bors commented Jan 26, 2017

📌 Commit 36ad34d has been approved by @eddyb

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Jan 28, 2017
save-analysis: get tables directly, accomodating them being missing

Fixes an ICE when running with save-analysis after an error

r? @eddyb
bors added a commit that referenced this pull request Jan 28, 2017
@bors bors merged commit 36ad34d into rust-lang:master Jan 28, 2017
@eddyb eddyb mentioned this pull request Apr 18, 2017
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants