derecated Acker::default()

Author: Keruspe

examples/publisher_confirms.rs

@@ -124,6 +124,7 @@ fn main() {
             .expect("publisher-confirms");
         assert!(confirm.is_ack());
         let message = confirm.take_message().unwrap();
+        let acker = message.delivery.acker.clone();
         assert_eq!(
             message,
             BasicReturnMessage {
@@ -134,7 +135,7 @@ fn main() {
                     redelivered: false,
                     properties: BasicProperties::default().with_priority(42),
                     data: payload.to_vec(),
-                    acker: Default::default(),
+                    acker,
                 },
                 reply_code: 312,
                 reply_text: "NO_ROUTE".into(),

src/acker.rs

@@ -7,12 +7,13 @@ use crate::{
     Error, Promise, PromiseResolver, Result,
 };
 
+use tracing::warn;
 use std::sync::{
     atomic::{AtomicBool, Ordering},
     Arc,
 };
 
-#[derive(Clone, Default, Debug)]
+#[derive(Clone, Debug)]
 pub struct Acker {
     channel_id: ChannelId,
     delivery_tag: DeliveryTag,
@@ -100,6 +101,14 @@ impl Acker {
     }
 }
 
+// FIXME: remove in 3.0
+impl Default for Acker {
+    fn default() -> Self {
+        warn!("Use of deprecated Acker::default() which provides an unusable Acker");
+        Self::new(0, 0, None, None)
+    }
+}
+
 impl PartialEq for Acker {
     fn eq(&self, other: &Acker) -> bool {
         self.channel_id == other.channel_id && self.delivery_tag == other.delivery_tag