From 1aba702ca1be0949bdefc5000e4af4b1b6ed6a8d Mon Sep 17 00:00:00 2001 From: Manoj Gupta Date: Thu, 26 Jan 2017 01:52:10 -0800 Subject: Fix libbrillo warnings to make it build with llvm-next on chromeos. Fix the warnings [-Wunused-lambda-capture] in libbrillo. Bug=chromium:685430 TEST=libbrillo compiles (cherry pick from chromium) Test: libbrillo compiles with newer llvm. Change-Id: Icb01a91f37b955cdf1e7beb93301951bd8385537 Reviewed-on: https://chromium-review.googlesource.com/433339 Commit-Ready: Manoj Gupta Tested-by: Manoj Gupta Reviewed-by: Luis Lozano Reviewed-by: Alex Deymo Reviewed-by: Alex Vakulenko --- brillo/http/http_request_unittest.cc | 2 +- brillo/message_loops/message_loop_unittest.cc | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/brillo/http/http_request_unittest.cc b/brillo/http/http_request_unittest.cc index edf056b..387c92d 100644 --- a/brillo/http/http_request_unittest.cc +++ b/brillo/http/http_request_unittest.cc @@ -178,7 +178,7 @@ TEST_F(HttpRequestTest, GetResponse) { }; auto finish_request_async = - [this, &read_data, &resp_data](const SuccessCallback& success_callback) { + [this, &read_data](const SuccessCallback& success_callback) { std::unique_ptr mock_stream{new MockStream}; EXPECT_CALL(*mock_stream, ReadBlocking(_, _, _, _)) .WillOnce(Invoke(read_data)) diff --git a/brillo/message_loops/message_loop_unittest.cc b/brillo/message_loops/message_loop_unittest.cc index 7cc2a30..0265fbf 100644 --- a/brillo/message_loops/message_loop_unittest.cc +++ b/brillo/message_loops/message_loop_unittest.cc @@ -282,7 +282,7 @@ TYPED_TEST(MessageLoopTest, DeleteAllPersistenIOTaskFromSelf) { task_ids[i] = this->loop_->WatchFileDescriptor( FROM_HERE, pipes[i].writer, MessageLoop::kWatchWrite, true /* persistent */, - Bind([this, kNumTasks, &task_ids] { + Bind([this, &task_ids] { for (int j = 0; j < kNumTasks; ++j) { // Once we cancel all the tasks, none should run, so this code runs // only once from one callback. -- cgit v1.2.3