commit a0ff2eccb34362a7c336284599b5bbbbebf49398
author Alex Vakulenko <avakulenko@google.com> Wed Mar 02 10:01:02 2016 -0800
committer Alex Vakulenko <avakulenko@google.com> Wed Mar 02 10:01:02 2016 -0800
tree 9386946f41b232951c02c1954977b9d7b4cf298c
parent 20f03a2966652cf76ea2e7449cebaedb6b457acc

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

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 @@
   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 @@
   }
 
   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));