Skip to content

Fix lint_cstring_as_ptr for expect #4362

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

Merged
merged 1 commit into from
Aug 9, 2019

Conversation

tesuji
Copy link
Contributor

@tesuji tesuji commented Aug 9, 2019

Closes #4312
changelog: none

@phansch phansch added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 9, 2019
@flip1995
Copy link
Member

flip1995 commented Aug 9, 2019

Thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Aug 9, 2019

📌 Commit c23a5c5 has been approved by flip1995

@bors
Copy link
Contributor

bors commented Aug 9, 2019

⌛ Testing commit c23a5c5 with merge 2f1f7b7...

bors added a commit that referenced this pull request Aug 9, 2019
Fix lint_cstring_as_ptr for expect

Closes #4312
changelog: none
@bors
Copy link
Contributor

bors commented Aug 9, 2019

💔 Test failed - checks-travis

@phansch
Copy link
Member

phansch commented Aug 9, 2019

@bors retry (spurious Travis windows build)

@bors
Copy link
Contributor

bors commented Aug 9, 2019

⌛ Testing commit c23a5c5 with merge 422ddc9...

bors added a commit that referenced this pull request Aug 9, 2019
Fix lint_cstring_as_ptr for expect

Closes #4312
changelog: none
@bors
Copy link
Contributor

bors commented Aug 9, 2019

💔 Test failed - checks-travis

@phansch
Copy link
Member

phansch commented Aug 9, 2019

@bors retry

@bors
Copy link
Contributor

bors commented Aug 9, 2019

⌛ Testing commit c23a5c5 with merge c55d38e...

bors added a commit that referenced this pull request Aug 9, 2019
Fix lint_cstring_as_ptr for expect

Closes #4312
changelog: none
@bors
Copy link
Contributor

bors commented Aug 9, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: flip1995
Pushing c55d38e to master...

@bors bors merged commit c23a5c5 into rust-lang:master Aug 9, 2019
@tesuji tesuji deleted the expect-on-cstring_as_ptr branch August 9, 2019 10:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties
Projects
None yet
Development

Successfully merging this pull request may close these issues.

temporary_cstring_as_ptr lint doesn't correctly trigger
4 participants