From f8987c393dbf7eb2b0b99144f6a98f3150a5c8bb Mon Sep 17 00:00:00 2001 From: Yurii Zubrytskyi Date: Wed, 22 Apr 2020 15:37:21 -0700 Subject: [PATCH] [cleanup] Fix a clang-tidy warning const parameters can't be moved but only get copied - removed const Bug: 153704006 Test: builds & boots Change-Id: If7e2250325bf1bc498afd3539f60bcb075a1d43b --- libutils/include/utils/Looper.h | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/libutils/include/utils/Looper.h b/libutils/include/utils/Looper.h index c439c5ce0..466fbb726 100644 --- a/libutils/include/utils/Looper.h +++ b/libutils/include/utils/Looper.h @@ -26,6 +26,8 @@ #include +#include + namespace android { /* @@ -438,9 +440,8 @@ private: struct MessageEnvelope { MessageEnvelope() : uptime(0) { } - MessageEnvelope(nsecs_t u, const sp h, - const Message& m) : uptime(u), handler(h), message(m) { - } + MessageEnvelope(nsecs_t u, sp h, const Message& m) + : uptime(u), handler(std::move(h)), message(m) {} nsecs_t uptime; sp handler;