Release 2.0.12

Clippy elidable_lifetime_names lint

.github/workflows/ci.yml

@@ -41,6 +41,7 @@ jobs:
       - run: cargo test --workspace --exclude thiserror_no_std_test
       - run: cargo test --manifest-path tests/no-std/Cargo.toml
         if: matrix.rust != '1.70.0'
+      - run: cargo test --no-default-features
       - uses: actions/upload-artifact@v4
         if: matrix.rust == 'nightly' && always()
         with:

.gitignore

@@ -1,3 +1,2 @@
-/target
-**/*.rs.bk
-Cargo.lock
+/target/
+/Cargo.lock

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "thiserror"
-version = "2.0.5"
+version = "2.0.12"
 authors = ["David Tolnay <dtolnay@gmail.com>"]
 categories = ["rust-patterns"]
 description = "derive(Error)"
@@ -28,7 +28,7 @@ default = ["std"]
 std = []
 
 [dependencies]
-thiserror-impl = { version = "=2.0.5", path = "impl" }
+thiserror-impl = { version = "=2.0.12", path = "impl" }
 
 [dev-dependencies]
 anyhow = "1.0.73"
@@ -41,4 +41,9 @@ members = ["impl", "tests/no-std"]
 
 [package.metadata.docs.rs]
 targets = ["x86_64-unknown-linux-gnu"]
-rustdoc-args = ["--generate-link-to-definition"]
+rustdoc-args = [
+    "--generate-link-to-definition",
+    "--extern-html-root-url=core=https://doc.rust-lang.org",
+    "--extern-html-root-url=alloc=https://doc.rust-lang.org",
+    "--extern-html-root-url=std=https://doc.rust-lang.org",
+]

impl/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "thiserror-impl"
-version = "2.0.5"
+version = "2.0.12"
 authors = ["David Tolnay <dtolnay@gmail.com>"]
 description = "Implementation detail of the `thiserror` crate"
 edition = "2021"
@@ -18,4 +18,10 @@ syn = "2.0.87"
 
 [package.metadata.docs.rs]
 targets = ["x86_64-unknown-linux-gnu"]
-rustdoc-args = ["--generate-link-to-definition"]
+rustdoc-args = [
+    "--generate-link-to-definition",
+    "--extern-html-root-url=core=https://doc.rust-lang.org",
+    "--extern-html-root-url=alloc=https://doc.rust-lang.org",
+    "--extern-html-root-url=std=https://doc.rust-lang.org",
+    "--extern-html-root-url=proc_macro=https://doc.rust-lang.org",
+]

impl/src/expand.rs

@@ -12,7 +12,7 @@ pub fn derive(input: &DeriveInput) -> TokenStream {
     match try_expand(input) {
         Ok(expanded) => expanded,
         // If there are invalid attributes in the input, expand to an Error impl
-        // anyway to minimize spurious knock-on errors in other code that uses
+        // anyway to minimize spurious secondary errors in other code that uses
         // this type as an Error.
         Err(error) => fallback::expand(input, error),
     }
@@ -169,16 +169,26 @@ fn impl_struct(input: Struct) -> TokenStream {
         let from = unoptional_type(from_field.ty);
         let source_var = Ident::new("source", span);
         let body = from_initializer(from_field, backtrace_field, &source_var);
-        quote_spanned! {span=>
-            #[allow(unused_qualifications, clippy::needless_lifetimes)]
+        let from_function = quote! {
+            fn from(#source_var: #from) -> Self {
+                #ty #body
+            }
+        };
+        let from_impl = quote_spanned! {span=>
             #[automatically_derived]
             impl #impl_generics ::core::convert::From<#from> for #ty #ty_generics #where_clause {
-                #[allow(deprecated)]
-                fn from(#source_var: #from) -> Self {
-                    #ty #body
-                }
+                #from_function
             }
-        }
+        };
+        Some(quote! {
+            #[allow(
+                deprecated,
+                unused_qualifications,
+                clippy::elidable_lifetime_names,
+                clippy::needless_lifetimes,
+            )]
+            #from_impl
+        })
     });
 
     if input.generics.type_params().next().is_some() {
@@ -434,15 +444,25 @@ fn impl_enum(input: Enum) -> TokenStream {
         let from = unoptional_type(from_field.ty);
         let source_var = Ident::new("source", span);
         let body = from_initializer(from_field, backtrace_field, &source_var);
-        Some(quote_spanned! {span=>
-            #[allow(unused_qualifications, clippy::needless_lifetimes)]
+        let from_function = quote! {
+            fn from(#source_var: #from) -> Self {
+                #ty::#variant #body
+            }
+        };
+        let from_impl = quote_spanned! {span=>
             #[automatically_derived]
             impl #impl_generics ::core::convert::From<#from> for #ty #ty_generics #where_clause {
-                #[allow(deprecated)]
-                fn from(#source_var: #from) -> Self {
-                    #ty::#variant #body
-                }
+                #from_function
             }
+        };
+        Some(quote! {
+            #[allow(
+                deprecated,
+                unused_qualifications,
+                clippy::elidable_lifetime_names,
+                clippy::needless_lifetimes,
+            )]
+            #from_impl
         })
     });
 

impl/src/fmt.rs

@@ -107,12 +107,14 @@ impl Display<'_> {
                 }
             };
             infinite_recursive |= member == *"self" && bound == Trait::Display;
-            if let Some(&field) = member_index.get(&member) {
-                implied_bounds.insert((field, bound));
-            } else {
-                out += &member.to_string();
-                continue;
-            }
+            let field = match member_index.get(&member) {
+                Some(&field) => field,
+                None => {
+                    out += &member.to_string();
+                    continue;
+                }
+            };
+            implied_bounds.insert((field, bound));
             let formatvar_prefix = if bonus_display {
                 "__display"
             } else if bound == Trait::Pointer {
@@ -129,15 +131,17 @@ impl Display<'_> {
             while user_named_args.contains(&formatvar) {
                 formatvar = IdentUnraw::new(format_ident!("_{}", formatvar.to_string()));
             }
+            formatvar.set_span(span);
             out += &formatvar.to_string();
             if !macro_named_args.insert(formatvar.clone()) {
                 // Already added to bindings by a previous use.
                 continue;
             }
-            let binding_value = match &member {
+            let mut binding_value = match &member {
                 MemberUnraw::Unnamed(index) => format_ident!("_{}", index),
                 MemberUnraw::Named(ident) => ident.to_local(),
             };
+            binding_value.set_span(span.resolved_at(fields[field].member.span()));
             let wrapped_binding_value = if bonus_display {
                 quote_spanned!(span=> #binding_value.as_display())
             } else if bound == Trait::Pointer {

impl/src/generics.rs

@@ -25,11 +25,12 @@ impl<'a> ParamsInScope<'a> {
 
 fn crawl(in_scope: &ParamsInScope, ty: &Type, found: &mut bool) {
     if let Type::Path(ty) = ty {
-        if ty.qself.is_none() {
-            if let Some(ident) = ty.path.get_ident() {
-                if in_scope.names.contains(ident) {
-                    *found = true;
-                }
+        if let Some(qself) = &ty.qself {
+            crawl(in_scope, &qself.ty, found);
+        } else {
+            let front = ty.path.segments.first().unwrap();
+            if front.arguments.is_none() && in_scope.names.contains(&front.ident) {
+                *found = true;
             }
         }
         for segment in &ty.path.segments {

impl/src/unraw.rs

@@ -28,6 +28,10 @@ impl IdentUnraw {
         }
         unraw
     }
+
+    pub fn set_span(&mut self, span: Span) {
+        self.0.set_span(span);
+    }
 }
 
 impl Display for IdentUnraw {

src/aserror.rs

@@ -44,7 +44,7 @@ impl<'a> AsDynError<'a> for dyn Error + Send + Sync + UnwindSafe + 'a {
 #[doc(hidden)]
 pub trait Sealed {}
 impl<T: Error> Sealed for T {}
-impl<'a> Sealed for dyn Error + 'a {}
-impl<'a> Sealed for dyn Error + Send + 'a {}
-impl<'a> Sealed for dyn Error + Send + Sync + 'a {}
-impl<'a> Sealed for dyn Error + Send + Sync + UnwindSafe + 'a {}
+impl Sealed for dyn Error + '_ {}
+impl Sealed for dyn Error + Send + '_ {}
+impl Sealed for dyn Error + Send + Sync + '_ {}
+impl Sealed for dyn Error + Send + Sync + UnwindSafe + '_ {}

src/lib.rs

@@ -9,8 +9,6 @@
 //! This library provides a convenient derive macro for the standard library's
 //! [`std::error::Error`] trait.
 //!
-//! [`std::error::Error`]: https://doc.rust-lang.org/std/error/trait.Error.html
-//!
 //! <br>
 //!
 //! # Example
@@ -259,8 +257,9 @@
 //!   [`anyhow`]: https://github.com/dtolnay/anyhow
 
 #![no_std]
-#![doc(html_root_url = "https://docs.rs/thiserror/2.0.5")]
+#![doc(html_root_url = "https://docs.rs/thiserror/2.0.12")]
 #![allow(
+    clippy::elidable_lifetime_names,
     clippy::module_name_repetitions,
     clippy::needless_lifetimes,
     clippy::return_self_not_must_use,

tests/test_backtrace.rs

@@ -1,3 +1,4 @@
+#![cfg(feature = "std")]
 #![cfg_attr(thiserror_nightly_testing, feature(error_generic_member_access))]
 
 use thiserror::Error;

tests/test_display.rs

@@ -1,4 +1,5 @@
 #![allow(
+    clippy::elidable_lifetime_names,
     clippy::needless_lifetimes,
     clippy::needless_raw_string_hashes,
     clippy::trivially_copy_pass_by_ref,
@@ -266,7 +267,7 @@ fn test_pointer() {
 }
 
 #[test]
-fn test_macro_rules() {
+fn test_macro_rules_variant_from_call_site() {
     // Regression test for https://github.com/dtolnay/thiserror/issues/86
 
     macro_rules! decl_error {
@@ -291,6 +292,30 @@ fn test_macro_rules() {
     assert("0", Error1::Repro(0));
 }
 
+#[test]
+fn test_macro_rules_message_from_call_site() {
+    // Regression test for https://github.com/dtolnay/thiserror/issues/398
+
+    macro_rules! decl_error {
+        ($($errors:tt)*) => {
+            #[derive(Error, Debug)]
+            pub enum Error {
+                $($errors)*
+            }
+        };
+    }
+
+    decl_error! {
+        #[error("{0}")]
+        Unnamed(u8),
+        #[error("{x}")]
+        Named { x: u8 },
+    }
+
+    assert("0", Error::Unnamed(0));
+    assert("0", Error::Named { x: 0 });
+}
+
 #[test]
 fn test_raw() {
     #[derive(Error, Debug)]

tests/test_expr.rs

@@ -1,6 +1,7 @@
 #![allow(clippy::iter_cloned_collect, clippy::uninlined_format_args)]
 
 use core::fmt::Display;
+#[cfg(feature = "std")]
 use std::path::PathBuf;
 use thiserror::Error;
 
@@ -90,6 +91,7 @@ fn test_rustup() {
 }
 
 // Regression test for https://github.com/dtolnay/thiserror/issues/335
+#[cfg(feature = "std")]
 #[test]
 #[allow(non_snake_case)]
 fn test_assoc_type_equality_constraint() {

tests/test_generics.rs

@@ -1,6 +1,7 @@
 #![allow(clippy::needless_late_init, clippy::uninlined_format_args)]
 
 use core::fmt::{self, Debug, Display};
+use core::str::FromStr;
 use thiserror::Error;
 
 pub struct NoFormat;
@@ -160,6 +161,24 @@ pub struct StructFromGeneric<E> {
 #[error(transparent)]
 pub struct StructTransparentGeneric<E>(pub E);
 
+// Should expand to:
+//
+//     impl<T: FromStr> Display for AssociatedTypeError<T>
+//     where
+//         T::Err: Display;
+//
+//     impl<T: FromStr> Error for AssociatedTypeError<T>
+//     where
+//         Self: Debug + Display;
+//
+#[derive(Error, Debug)]
+pub enum AssociatedTypeError<T: FromStr> {
+    #[error("couldn't parse matrix")]
+    Other,
+    #[error("couldn't parse entry: {0}")]
+    EntryParseError(T::Err),
+}
+
 // Regression test for https://github.com/dtolnay/thiserror/issues/345
 #[test]
 fn test_no_bound_on_named_fmt() {

tests/test_lints.rs

@@ -4,6 +4,17 @@ use thiserror::Error;
 
 pub use std::error::Error;
 
+#[test]
+fn test_allow_attributes() {
+    #![deny(clippy::allow_attributes)]
+
+    #[derive(Error, Debug)]
+    #[error("...")]
+    pub struct MyError(#[from] anyhow::Error);
+
+    let _: MyError;
+}
+
 #[test]
 fn test_unused_qualifications() {
     #![deny(unused_qualifications)]
@@ -22,7 +33,7 @@ fn test_unused_qualifications() {
 #[test]
 fn test_needless_lifetimes() {
     #![allow(dead_code)]
-    #![deny(clippy::needless_lifetimes)]
+    #![deny(clippy::elidable_lifetime_names, clippy::needless_lifetimes)]
 
     #[derive(Error, Debug)]
     #[error("...")]
@@ -64,6 +75,16 @@ fn test_deprecated() {
         Variant,
     }
 
+    #[derive(Error, Debug)]
+    pub enum DeprecatedFrom {
+        #[error(transparent)]
+        Variant(
+            #[from]
+            #[allow(deprecated)]
+            DeprecatedStruct,
+        ),
+    }
+
     #[allow(deprecated)]
     let _: DeprecatedStruct;
     #[allow(deprecated)]

tests/test_option.rs

@@ -1,3 +1,4 @@
+#![cfg(feature = "std")]
 #![cfg_attr(thiserror_nightly_testing, feature(error_generic_member_access))]
 
 #[cfg(thiserror_nightly_testing)]

tests/test_path.rs

@@ -1,3 +1,5 @@
+#![cfg(feature = "std")]
+
 use core::fmt::Display;
 use ref_cast::RefCast;
 use std::path::{Path, PathBuf};

tests/ui/same-from-type.rs

@@ -5,7 +5,7 @@ pub enum Error {
     #[error("failed to open")]
     OpenFile(#[from] std::io::Error),
     #[error("failed to close")]
-    CloseFIle(#[from] std::io::Error),
+    CloseFile(#[from] std::io::Error),
 }
 
 fn main() {}

tests/ui/same-from-type.stderr

@@ -4,5 +4,5 @@ error[E0119]: conflicting implementations of trait `From<std::io::Error>` for ty
 6 |     OpenFile(#[from] std::io::Error),
   |              ------- first implementation here
 7 |     #[error("failed to close")]
-8 |     CloseFIle(#[from] std::io::Error),
+8 |     CloseFile(#[from] std::io::Error),
   |               ^^^^^^^ conflicting implementation for `Error`