From 31021a8c68ec25454bde7d9419fc721548132cda Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Emilio=20Cobos=20=C3=81lvarez?= <emilio@crisal.io>
Date: Thu, 26 Dec 2024 20:11:48 +0100
Subject: [PATCH] zb, zm: Don't require an owned header for property accesses.

This avoids a useless clone, see also #1179.
---
 zbus/tests/e2e.rs        | 4 ++--
 zbus_macros/src/iface.rs | 4 +---
 2 files changed, 3 insertions(+), 5 deletions(-)

diff --git a/zbus/tests/e2e.rs b/zbus/tests/e2e.rs
index 7419f528a..f989f4bc4 100644
--- a/zbus/tests/e2e.rs
+++ b/zbus/tests/e2e.rs
@@ -266,7 +266,7 @@ impl MyIface {
     #[zbus(property)]
     fn test_header_prop(
         &self,
-        #[zbus(header)] header: Option<Header<'_>>,
+        #[zbus(header)] header: Option<&Header<'_>>,
         #[zbus(connection)] connection: &Connection,
         #[zbus(object_server)] object_server: &ObjectServer,
         #[zbus(signal_emitter)] emitter: SignalEmitter<'_>,
@@ -283,7 +283,7 @@ impl MyIface {
     fn set_test_header_prop(
         &self,
         value: bool,
-        #[zbus(header)] header: Option<Header<'_>>,
+        #[zbus(header)] header: Option<&Header<'_>>,
         #[zbus(connection)] connection: &Connection,
         #[zbus(object_server)] object_server: &ObjectServer,
         #[zbus(signal_emitter)] emitter: SignalEmitter<'_>,
diff --git a/zbus_macros/src/iface.rs b/zbus_macros/src/iface.rs
index d29def692..3c6b81970 100644
--- a/zbus_macros/src/iface.rs
+++ b/zbus_macros/src/iface.rs
@@ -1038,9 +1038,7 @@ fn get_args_from_inputs(
                 let header_arg = &input.pat;
 
                 header_arg_decl = match method_type {
-                    MethodType::Property(_) => Some(quote! {
-                        let #header_arg = ::std::option::Option::<&#zbus::message::Header<'_>>::cloned(header);
-                    }),
+                    MethodType::Property(_) => Some(quote! { let #header_arg = header; }),
                     _ => Some(quote! { let #header_arg = message.header(); }),
                 };
             } else if signal_context || signal_emitter {