msg/Message: remove unused local variables

and avoid creating a temporary instance of `ceph_msg_footer`.

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/msg/Message.cc b/src/msg/Message.cc
index 802c73d7a6314f2329aa97b84aa1115c6d7d8d26..50982e6181dae5e9ef527c0740708c7e1b7a565b 100644 (file)
--- a/src/msg/Message.cc
+++ b/src/msg/Message.cc
@@ -957,15 +957,12 @@ void Message::decode_trace(bufferlist::const_iterator &p, bool create)
 
 void encode_message(Message *msg, uint64_t features, bufferlist& payload)
 {
-  bufferlist front, middle, data;
   ceph_msg_footer_old old_footer;
-  ceph_msg_footer footer;
   msg->encode(features, MSG_CRC_ALL);
   encode(msg->get_header(), payload);
 
   // Here's where we switch to the old footer format.  PLR
-
-  footer = msg->get_footer();
+  ceph_msg_footer footer = msg->get_footer();
   old_footer.front_crc = footer.front_crc;   
   old_footer.middle_crc = footer.middle_crc;   
   old_footer.data_crc = footer.data_crc;