about summary refs log tree commit diff
diff options
context:
space:
mode:
authorMattias Wallin <mattias@klingawallin.se>2024-09-15 12:07:07 +0200
committerYacin Tmimi <yacintmimi@gmail.com>2024-09-16 09:29:53 -0400
commit5230e8fbe672321d43438fad0d0c00d4118e566e (patch)
tree8fc77c411b8be85dd7a0d4e06949e53839d01383
parentd6d9e76240cdc8168c91d7bd89de37ef863b87bb (diff)
downloadrust-5230e8fbe672321d43438fad0d0c00d4118e566e.tar.gz
rust-5230e8fbe672321d43438fad0d0c00d4118e566e.zip
Non-panicking Display for Path
-rw-r--r--src/bin/main.rs4
-rw-r--r--src/config/file_lines.rs2
-rw-r--r--src/test/mod.rs12
3 files changed, 7 insertions, 11 deletions
diff --git a/src/bin/main.rs b/src/bin/main.rs
index 0d593c6c416..c7d3a060d54 100644
--- a/src/bin/main.rs
+++ b/src/bin/main.rs
@@ -341,10 +341,10 @@ fn format(
 
     for file in files {
         if !file.exists() {
-            eprintln!("Error: file `{}` does not exist", file.to_str().unwrap());
+            eprintln!("Error: file `{}` does not exist", file.display());
             session.add_operational_error();
         } else if file.is_dir() {
-            eprintln!("Error: `{}` is a directory", file.to_str().unwrap());
+            eprintln!("Error: `{}` is a directory", file.display());
             session.add_operational_error();
         } else {
             // Check the file directory if the config-path could not be read or not provided
diff --git a/src/config/file_lines.rs b/src/config/file_lines.rs
index 7f32765195d..c53ec6371e9 100644
--- a/src/config/file_lines.rs
+++ b/src/config/file_lines.rs
@@ -38,7 +38,7 @@ impl From<rustc_span::FileName> for FileName {
 impl fmt::Display for FileName {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
         match self {
-            FileName::Real(p) => write!(f, "{}", p.to_str().unwrap()),
+            FileName::Real(p) => write!(f, "{}", p.display()),
             FileName::Stdin => write!(f, "<stdin>"),
         }
     }
diff --git a/src/test/mod.rs b/src/test/mod.rs
index 28f95ee54e5..d62da08fff8 100644
--- a/src/test/mod.rs
+++ b/src/test/mod.rs
@@ -105,10 +105,9 @@ fn is_file_skip(path: &Path) -> bool {
 fn get_test_files(path: &Path, recursive: bool) -> Vec<PathBuf> {
     let mut files = vec![];
     if path.is_dir() {
-        for entry in fs::read_dir(path).expect(&format!(
-            "couldn't read directory {}",
-            path.to_str().unwrap()
-        )) {
+        for entry in
+            fs::read_dir(path).expect(&format!("couldn't read directory {}", path.display()))
+        {
             let entry = entry.expect("couldn't get `DirEntry`");
             let path = entry.path();
             if path.is_dir() && recursive {
@@ -122,10 +121,7 @@ fn get_test_files(path: &Path, recursive: bool) -> Vec<PathBuf> {
 }
 
 fn verify_config_used(path: &Path, config_name: &str) {
-    for entry in fs::read_dir(path).expect(&format!(
-        "couldn't read {} directory",
-        path.to_str().unwrap()
-    )) {
+    for entry in fs::read_dir(path).expect(&format!("couldn't read {} directory", path.display())) {
         let entry = entry.expect("couldn't get directory entry");
         let path = entry.path();
         if path.extension().map_or(false, |f| f == "rs") {