Apply suggestions from code review

lolbinarycat · ibraheemdev · lolbinarycat · commit f4e0bf8101e1 · 2025-08-10T12:40:41.000-05:00
Co-authored-by: Ibraheem Ahmed &lt;ibraheem@ibraheem.ca&gt;
diff --git a/library/std/src/fs.rs b/library/std/src/fs.rs
@@ -3134,9 +3134,9 @@ pub fn set_permissions<P: AsRef<Path>>(path: P, perm: Permissions) -> io::Result
 ///
 /// # Platform-specific behavior
 ///
-/// Currently unimplmented on windows.
+/// Currently unimplemented on Windows.
 ///
-/// On unix platforms, this results in a [`FilesystemLoop`] error if the last element is a symlink.
+/// On Unix platforms, this results in a [`FilesystemLoop`] error if the last element is a symlink.
 ///
 /// This behavior may change in the future.
 ///
diff --git a/library/std/src/sys/fs/mod.rs b/library/std/src/sys/fs/mod.rs
@@ -118,7 +118,9 @@ pub fn set_permissions_nofollow(path: &Path, perm: crate::fs::Permissions) -> io
 
 #[cfg(not(unix))]
 pub fn set_permissions_nofollow(_path: &Path, _perm: crate::fs::Permissions) -> io::Result<()> {
-    crate::unimplemented!("set_permissions_nofollow is not implmented on non-unix platforms")
+    crate::unimplemented!(
+        "`set_permissions_nofollow` is currently only implemented on Unix platforms"
+    )
 }
 
 pub fn canonicalize(path: &Path) -> io::Result<PathBuf> {

Original file line number	Diff line number	Diff line change
`@@ -118,7 +118,9 @@ pub fn set_permissions_nofollow(path: &Path, perm: crate::fs::Permissions) -> io`
`118`	`118`
`119`	`119`	`#[cfg(not(unix))]`
`120`	`120`	`pub fn set_permissions_nofollow(_path: &Path, _perm: crate::fs::Permissions) -> io::Result<()> {`
`121`		`- crate::unimplemented!("set_permissions_nofollow is not implmented on non-unix platforms")`
	`121`	`+ crate::unimplemented!(`
	`122`	+ "`set_permissions_nofollow` is currently only implemented on Unix platforms"
	`123`	`+ )`
`122`	`124`	`}`
`123`	`125`
`124`	`126`	`pub fn canonicalize(path: &Path) -> io::Result<PathBuf> {`