Rust: Make the path in Translator a PathBuf

Author: paldepind

rust/extractor/src/main.rs

@@ -64,12 +64,11 @@ impl<'a> Extractor<'a> {
 
         let before_extract = Instant::now();
         let line_index = LineIndex::new(text.as_ref());
-        let display_path = file.to_string_lossy();
         let mut trap = self.traps.create("source", file);
         let label = trap.emit_file(file);
         let mut translator = translate::Translator::new(
             trap,
-            display_path.as_ref(),
+            file,
             label,
             line_index,
             semantics_info.as_ref().ok(),
@@ -98,7 +97,7 @@ impl<'a> Extractor<'a> {
         translator.trap.commit().unwrap_or_else(|err| {
             error!(
                 "Failed to write trap file for: {}: {}",
-                display_path,
+                file.display(),
                 err.to_string()
             )
         });

rust/extractor/src/translate/base.rs

@@ -21,6 +21,7 @@ use ra_ap_syntax::{
 use ra_ap_syntax_bridge::{
     DocCommentDesugarMode, syntax_node_to_token_tree, token_tree_to_syntax_node,
 };
+use std::path::{Path, PathBuf};
 
 impl Emission<ast::Item> for Translator<'_> {
     fn pre_emit(&mut self, node: &ast::Item) -> Option<Label<generated::Item>> {
@@ -125,7 +126,7 @@ pub enum SourceKind {
 
 pub struct Translator<'db> {
     pub trap: TrapFile,
-    path: String,
+    path: PathBuf,
     label: Label<generated::File>,
     line_index: LineIndex,
     file_id: Option<EditionedFileId>,
@@ -147,15 +148,15 @@ const DIAGNOSTIC_LIMIT_PER_FILE: usize = 100;
 impl<'db> Translator<'db> {
     pub fn new(
         trap: TrapFile,
-        path: &str,
+        path: &Path,
         label: Label<generated::File>,
         line_index: LineIndex,
         semantic_info: Option<&FileSemanticInformation<'db>>,
         source_kind: SourceKind,
     ) -> Translator<'db> {
         Translator {
             trap,
-            path: path.to_owned(),
+            path: path.to_path_buf(),
             label,
             line_index,
             file_id: semantic_info.map(|i| i.file_id),
@@ -293,7 +294,7 @@ impl<'db> Translator<'db> {
         dispatch_to_tracing!(
             severity,
             "{}:{}:{}: {}",
-            self.path,
+            self.path.display(),
             start.line + 1,
             start.col + 1,
             &full_message,