From a0ff2eccb34362a7c336284599b5bbbbebf49398 Mon Sep 17 00:00:00 2001 From: Alex Vakulenko Date: Wed, 2 Mar 2016 10:01:02 -0800 Subject: Remove clang warning in unit tests The new clang compiler compains about |state| and |origin| possibly being uninitialized if StringToEnum fails. There is no harm of initializing them to some default values... BUG: 27437721 Change-Id: If8062a337d5530223fcab6480bbbeab257893d08 --- buffet/binder_command_proxy_unittest.cc | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/buffet/binder_command_proxy_unittest.cc b/buffet/binder_command_proxy_unittest.cc index 0905440..2ba083a 100644 --- a/buffet/binder_command_proxy_unittest.cc +++ b/buffet/binder_command_proxy_unittest.cc @@ -84,7 +84,7 @@ class BinderCommandProxyTest : public ::testing::Test { BinderCommandProxy* GetCommandProxy() const { return proxy_.get(); } weave::Command::State GetCommandState() const { - weave::Command::State state; + weave::Command::State state = weave::Command::State::kAborted; android::String16 state_string; EXPECT_TRUE(GetCommandProxy()->getState(&state_string).isOk()); EXPECT_TRUE(StringToEnum(ToString(state_string), &state)); @@ -92,7 +92,7 @@ class BinderCommandProxyTest : public ::testing::Test { } weave::Command::Origin GetCommandOrigin() const { - weave::Command::Origin origin; + weave::Command::Origin origin = weave::Command::Origin::kCloud; android::String16 origin_string; EXPECT_TRUE(GetCommandProxy()->getOrigin(&origin_string).isOk()); EXPECT_TRUE(StringToEnum(ToString(origin_string), &origin)); -- cgit v1.2.3