Skip to content

Sync io - #1247

Closed
boggle wants to merge 4 commits into
rust-lang:masterfrom
boggle:sync_io
Closed

Sync io#1247
boggle wants to merge 4 commits into
rust-lang:masterfrom
boggle:sync_io

Conversation

@boggle

Copy link
Copy Markdown
Contributor

Adds fsync and fflush calls to io.

  • fsync tries to do something reasonable on mac, linux depending on fsync::level
  • on win32, it just calls fsync (dont know what to do there)
  • needs testing

@brson

brson commented Dec 2, 2011

Copy link
Copy Markdown
Contributor

This looks good to me. If you don't mind I will squash these down to a single commit. Waiting for the try build to finish now.

@boggle

Copy link
Copy Markdown
ContributorAuthor

Sure, that's ok.

@brson

brson commented Dec 2, 2011

Copy link
Copy Markdown
Contributor

No fsync on windows. I just replaced it with fail and opened #1253. Thanks!

@brsonbrson closed this Dec 2, 2011
JohnTitor pushed a commit to JohnTitor/rust that referenced this pull request Jul 26, 2022
coastalwhite pushed a commit to coastalwhite/rust that referenced this pull request Aug 5, 2023
celinval pushed a commit to celinval/rust-dev that referenced this pull request Jun 4, 2024
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
1247: Remove collect r=matklad a=kjeremy
Might help with perf
Co-authored-by: Jeremy Kolb <kjeremy@gmail.com>
U007D pushed a commit to U007D/rust-mos that referenced this pull request Aug 21, 2026
call error location was fixed by rustup
Sign up for freeto 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.

2 participants

@boggle@brson