Skip to content
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

Add stat method to std::io::fs::File to stat without a Path. #14128

Closed
wants to merge 1 commit into from

Conversation

yuriks
Copy link
Contributor

@yuriks yuriks commented May 12, 2014

The FileStat struct contained a path field, which was filled by the
stat and lstat function. Since this field isn't in fact returned by
the operating system (it was copied from the paths passed to the
functions) it was removed, as in the fstat case we aren't working with
a Path, but directly with a fd.

If your code used the path field of FileStat you will now have to
manually store the path passed to stat along with the returned struct.

[breaking-change]

The `FileStat` struct contained a `path` field, which was filled by the
`stat` and `lstat` function. Since this field isn't in fact returned by
the operating system (it was copied from the paths passed to the
functions) it was removed, as in the `fstat` case we aren't working with
a `Path`, but directly with a fd.

If your code used the `path` field of `FileStat` you will now have to
manually store the path passed to `stat` along with the returned struct.

[breaking-change]
@yuriks yuriks deleted the fstat branch May 13, 2014 15:18
bors added a commit to rust-lang-ci/rust that referenced this pull request Feb 20, 2023
internal: Improve parser recovery for delimited lists

Closes rust-lang/rust-analyzer#11188, rust-lang/rust-analyzer#10410, rust-lang/rust-analyzer#10173

Should probably be merged after the stable release as this might get the parser stuck if I missed something
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.

2 participants