From 6cefcb0b54a52c5507362d3ae33bc11843580ae1 Mon Sep 17 00:00:00 2001 From: Tim Hutt Date: Mon, 7 Oct 2019 13:32:07 +0100 Subject: [PATCH 1/2] Add File::with_options This provides a more fluent API to create files with options, and also avoids the need to import OpenOptions. --- src/libstd/fs.rs | 35 +++++++++++++++++++++++++++++++++-- 1 file changed, 33 insertions(+), 2 deletions(-) diff --git a/src/libstd/fs.rs b/src/libstd/fs.rs index fc26dcb321148..ce1680fde2236 100644 --- a/src/libstd/fs.rs +++ b/src/libstd/fs.rs @@ -29,7 +29,7 @@ use crate::time::SystemTime; /// /// # Examples /// -/// Creates a new file and write bytes to it: +/// Creates a new file and write bytes to it (you can also use [`std::fs::write`]): /// /// ```no_run /// use std::fs::File; @@ -42,7 +42,7 @@ use crate::time::SystemTime; /// } /// ``` /// -/// Read the contents of a file into a [`String`]: +/// Read the contents of a file into a [`String`] (you can also use [`std::fs::read`]): /// /// ```no_run /// use std::fs::File; @@ -397,6 +397,37 @@ impl File { OpenOptions::new().write(true).create(true).truncate(true).open(path.as_ref()) } + /// Returns a new OpenOptions object. + /// + /// This function returns a new OpenOptions object that you can use to + /// open or create a file with specific options if `open()` or `create()` + /// are not appropriate. + /// + /// It is equivalent to `OpenOptions::new()` but allows you to write more + /// readable code. Instead of `OpenOptions::new().read(true).open("foo.txt")` + /// you can write `File::with_options().read(true).open("foo.txt"). This + /// also avoids the need to import `OpenOptions`. + /// + /// See the [`OpenOptions::new`] function for more details. + /// + /// [`OpenOptions::new`]: struct.OpenOptions.html#method.new + /// + /// # Examples + /// + /// ```no_run + /// #![feature(with_options)] + /// use std::fs::File; + /// + /// fn main() -> std::io::Result<()> { + /// let mut f = File::with_options().read(true).open("foo.txt")?; + /// Ok(()) + /// } + /// ``` + #[unstable(feature = "with_options", issue = "65439")] + pub fn with_options() -> OpenOptions { + OpenOptions::new() + } + /// Attempts to sync all OS-internal metadata to disk. /// /// This function will attempt to ensure that all in-memory data reaches the From c8150cb1594af7f3c2fe4b3968809637ca274260 Mon Sep 17 00:00:00 2001 From: Tim Hutt Date: Wed, 16 Oct 2019 09:03:52 +0100 Subject: [PATCH 2/2] Fix read/write links hopefully! --- src/libstd/fs.rs | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/libstd/fs.rs b/src/libstd/fs.rs index ce1680fde2236..1c7efd2f5320a 100644 --- a/src/libstd/fs.rs +++ b/src/libstd/fs.rs @@ -29,7 +29,7 @@ use crate::time::SystemTime; /// /// # Examples /// -/// Creates a new file and write bytes to it (you can also use [`std::fs::write`]): +/// Creates a new file and write bytes to it (you can also use [`write`]): /// /// ```no_run /// use std::fs::File; @@ -42,7 +42,7 @@ use crate::time::SystemTime; /// } /// ``` /// -/// Read the contents of a file into a [`String`] (you can also use [`std::fs::read`]): +/// Read the contents of a file into a [`String`] (you can also use [`read`]): /// /// ```no_run /// use std::fs::File; @@ -89,6 +89,8 @@ use crate::time::SystemTime; /// [`Write`]: ../io/trait.Write.html /// [`BufReader`]: ../io/struct.BufReader.html /// [`sync_all`]: struct.File.html#method.sync_all +/// [`read`]: fn.read.html +/// [`write`]: fn.write.html #[stable(feature = "rust1", since = "1.0.0")] pub struct File { inner: fs_imp::File,