Cleanup internals

Cargo.toml

@@ -22,7 +22,7 @@ thiserror = "1.0"
 trybuild = { version = "1.0.19", features = ["diff"] }
 backtrace = "0.3.46"
 anyhow = "1.0.28"
-syn = { version = "1.0", features = ["full"] }
+syn = { version = "2.0", features = ["full"] }
 pyo3 = { version = "0.13", default-features = false, features = ["auto-initialize"] }
 
 [dependencies]

src/lib.rs

@@ -329,14 +329,13 @@
 //! [`color-spantrace`]: https://github.com/yaahc/color-spantrace
 //! [`color-backtrace`]: https://github.com/athre0z/color-backtrace
 #![doc(html_root_url = "https://docs.rs/eyre/0.6.8")]
 #![warn(
     missing_debug_implementations,
     missing_docs,
     missing_doc_code_examples,
     rust_2018_idioms,
     unreachable_pub,
     bad_style,
-    const_err,
     dead_code,
     improper_ctypes,
     non_shorthand_field_patterns,

src/macros.rs

@@ -40,7 +40,6 @@
 ///
 /// # fn main() -> Result<()> {
 /// #     let depth = 0;
-/// #     let err: &'static dyn std::error::Error = &ScienceError::RecursionLimitExceeded;
 /// #
 /// if depth > MAX_DEPTH {
 ///     bail!(ScienceError::RecursionLimitExceeded);

tests/test_context.rs

@@ -21,7 +21,7 @@ macro_rules! context_type {
         #[derive(Debug)]
         struct $name {
             message: &'static str,
-            drop: DetectDrop,
+            _drop: DetectDrop,
         }
 
         impl Display for $name {
@@ -74,15 +74,15 @@ fn make_chain() -> (Report, Dropped) {
     let mid = Err::<(), LowLevel>(low)
         .wrap_err(MidLevel {
             message: "failed to load config",
-            drop: DetectDrop::new(&dropped.mid),
+            _drop: DetectDrop::new(&dropped.mid),
         })
         .unwrap_err();
 
     // impl Report for Result<T, Error>
     let high = Err::<(), Report>(mid)
         .wrap_err(HighLevel {
             message: "failed to start server",
-            drop: DetectDrop::new(&dropped.high),
+            _drop: DetectDrop::new(&dropped.high),
         })
         .unwrap_err();
 

tests/test_macros.rs

@@ -62,15 +62,15 @@ fn test_temporaries() {
         // time it's done evaluating, those will stick around until the
         // semicolon, which is on the other side of the await point, making the
         // enclosing future non-Send.
-        Ready(Some(eyre!("..."))).await;
+        let _ = Ready(Some(eyre!("..."))).await;
     });
 
     fn message(cell: Cell<&str>) -> &str {
         cell.get()
     }
 
     require_send_sync(async {
-        Ready(Some(eyre!(message(Cell::new("..."))))).await;
+        let _ = Ready(Some(eyre!(message(Cell::new("..."))))).await;
     });
 }