Use name instead of absolute path for diagnostics.

Author: reitermarkus

bindgen/codegen/mod.rs

@@ -4385,12 +4385,13 @@ fn unsupported_abi_diagnostic(
             ..
         }) = location.cloned()
         {
-            let file_path = file.path();
-            if let Ok(Some(source)) = get_line(&file_path, line) {
+            if let Ok(Some(source)) = get_line(file.path(), line) {
                 let mut slice = Slice::default();
-                slice
-                    .with_source(source)
-                    .with_location(file_path, line, column);
+                slice.with_source(source).with_location(
+                    file.name(),
+                    line,
+                    column,
+                );
                 diag.add_slice(slice);
             }
         }
@@ -4427,12 +4428,13 @@ fn variadic_fn_diagnostic(
             ..
         }) = location.cloned()
         {
-            let file_path = file.path();
-            if let Ok(Some(source)) = get_line(&file_path, line) {
+            if let Ok(Some(source)) = get_line(file.path(), line) {
                 let mut slice = Slice::default();
-                slice
-                    .with_source(source)
-                    .with_location(file_path, line, column);
+                slice.with_source(source).with_location(
+                    file.name(),
+                    line,
+                    column,
+                );
                 diag.add_slice(slice);
             }
         }

bindgen/diagnostics.rs

@@ -2,6 +2,7 @@
 //!
 //! The entry point of this module is the [`Diagnostic`] type.
 
+use std::fmt::Write;
 use std::io::{self, BufRead, BufReader};
 use std::path::Path;
 use std::{borrow::Cow, fs::File};
@@ -110,7 +111,7 @@ impl<'a> Diagnostic<'a> {
                 slices.push(ExtSlice {
                     source: source.as_ref(),
                     line_start: slice.line.unwrap_or_default(),
-                    origin: slice.filename.as_deref(),
+                    origin: slice.origin.as_deref(),
                     annotations: vec![],
                     fold: false,
                 })
@@ -146,9 +147,9 @@ impl<'a> Diagnostic<'a> {
 /// A slice of source code.
 #[derive(Default)]
 pub(crate) struct Slice<'a> {
-    source: Option<Cow<'a, str>>,
-    filename: Option<String>,
     line: Option<usize>,
+    origin: Option<String>,
+    source: Option<Cow<'a, str>>,
 }
 
 impl<'a> Slice<'a> {
@@ -162,15 +163,14 @@ impl<'a> Slice<'a> {
     }
 
     /// Set the file, line and column.
-    pub(crate) fn with_location<P: AsRef<Path>>(
+    pub(crate) fn with_location(
         &mut self,
-        path: P,
+        file_name: &str,
         line: usize,
-        col: usize,
+        column: usize,
     ) -> &mut Self {
-        self.filename =
-            Some(format!("{}:{}:{}", path.as_ref().display(), line, col));
         self.line = Some(line);
+        self.origin = Some(format!("{file_name}:{line}:{column}"));
         self
     }
 }

bindgen/ir/var.rs

@@ -474,11 +474,10 @@ fn duplicated_macro_diagnostic(
             file, line, column, ..
         } = location
         {
-            let file_path = file.path();
-            if let Ok(Some(code)) = get_line(&file_path, line) {
+            if let Ok(Some(code)) = get_line(file.path(), line) {
                 source = code.into();
             }
-            slice.with_location(file_path, line, column);
+            slice.with_location(file.name(), line, column);
         }
 
         slice.with_source(source);