Rollup merge of #43204 - jackpot51:patch-3, r=alexcrichton

steveklabnik · web-flow · commit da3f5b80ed8a · 2017-07-13T10:45:22.000-04:00
Implement fs::rename in sys::redox

This uses a simple implementation of copy + unlink. Redox does not have a rename or link system call for a faster implementation.
diff --git a/src/libstd/sys/redox/fs.rs b/src/libstd/sys/redox/fs.rs
@@ -383,9 +383,10 @@ pub fn unlink(p: &Path) -> io::Result<()> {
     Ok(())
 }
 
-pub fn rename(_old: &Path, _new: &Path) -> io::Result<()> {
-    ::sys_common::util::dumb_print(format_args!("Rename\n"));
-    unimplemented!();
+pub fn rename(old: &Path, new: &Path) -> io::Result<()> {
+    copy(old, new)?;
+    unlink(old)?;
+    Ok(())
 }
 
 pub fn set_perm(p: &Path, perm: FilePermissions) -> io::Result<()> {

Original file line number	Diff line number	Diff line change
`@@ -383,9 +383,10 @@ pub fn unlink(p: &Path) -> io::Result<()> {`
`383`	`383`	`Ok(())`
`384`	`384`	`}`
`385`	`385`
`386`		`-pub fn rename(_old: &Path, _new: &Path) -> io::Result<()> {`
`387`		`- ::sys_common::util::dumb_print(format_args!("Rename\n"));`
`388`		`- unimplemented!();`
	`386`	`+pub fn rename(old: &Path, new: &Path) -> io::Result<()> {`
	`387`	`+ copy(old, new)?;`
	`388`	`+ unlink(old)?;`
	`389`	`+ Ok(())`
`389`	`390`	`}`
`390`	`391`
`391`	`392`	`pub fn set_perm(p: &Path, perm: FilePermissions) -> io::Result<()> {`