Skip to content

Add shims for file handling#962

Merged
bors merged 12 commits intorust-lang:masterfrom
pvdrz:file-shim
Oct 1, 2019
Merged

Add shims for file handling#962
bors merged 12 commits intorust-lang:masterfrom
pvdrz:file-shim

Conversation

@pvdrz
Copy link
Copy Markdown
Contributor

@pvdrz pvdrz commented Sep 25, 2019

This adds the bare minimum to be able to do File::open and File::read. I also need some feedback about how to handle certain things

Comment thread src/shims/io.rs Outdated
Comment thread src/shims/io.rs
Comment thread src/shims/io.rs Outdated
Comment thread tests/hello.txt
@pvdrz pvdrz changed the title [WIP] Add shims for file handling Add shims for file handling Sep 25, 2019
Comment thread src/shims/io.rs
@bors
Copy link
Copy Markdown
Contributor

bors commented Sep 28, 2019

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

Comment thread src/shims/io.rs Outdated
Comment thread src/shims/io.rs Outdated
Comment thread src/shims/io.rs Outdated
Comment thread src/shims/io.rs Outdated
Comment thread src/shims/io.rs Outdated
Comment thread src/shims/io.rs Outdated
Comment thread src/shims/io.rs Outdated
@pvdrz pvdrz requested a review from oli-obk October 1, 2019 14:19
Comment thread src/shims/io.rs
@oli-obk
Copy link
Copy Markdown
Contributor

oli-obk commented Oct 1, 2019

@bors r+

@bors
Copy link
Copy Markdown
Contributor

bors commented Oct 1, 2019

📌 Commit d0509d7 has been approved by oli-obk

bors added a commit that referenced this pull request Oct 1, 2019
Add shims for file handling

This adds the bare minimum to be able to do `File::open` and `File::read`. I also need some feedback about how to handle certain things
@bors
Copy link
Copy Markdown
Contributor

bors commented Oct 1, 2019

⌛ Testing commit d0509d7 with merge 1037f69...

@bors
Copy link
Copy Markdown
Contributor

bors commented Oct 1, 2019

☀️ Test successful - checks-travis, status-appveyor
Approved by: oli-obk
Pushing 1037f69 to master...

@bors bors merged commit d0509d7 into rust-lang:master Oct 1, 2019
Comment thread src/shims/foreign_items.rs
Comment thread src/lib.rs
Comment thread src/shims/io.rs
Comment thread src/shims/io.rs
Comment thread src/shims/io.rs
Comment thread src/shims/io.rs
Comment thread tests/run-pass/file_read.rs
Comment thread src/shims/io.rs
@RalfJung RalfJung mentioned this pull request Oct 11, 2019
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.

4 participants