diff options
| author | Manoj Gupta <manojgupta@google.com> | 2017-01-26 01:52:10 -0800 |
|---|---|---|
| committer | Manoj Gupta <manojgupta@google.com> | 2017-01-30 12:17:02 -0800 |
| commit | 1aba702ca1be0949bdefc5000e4af4b1b6ed6a8d (patch) | |
| tree | 9b1b6308de5db4aff27b142d93a230d25fd1b303 | |
| parent | b167bbb724bdac160bf94e86f2a9a2167ad51e84 (diff) | |
| download | platform_external_libbrillo-1aba702ca1be0949bdefc5000e4af4b1b6ed6a8d.tar.gz platform_external_libbrillo-1aba702ca1be0949bdefc5000e4af4b1b6ed6a8d.tar.bz2 platform_external_libbrillo-1aba702ca1be0949bdefc5000e4af4b1b6ed6a8d.zip | |
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 <manojgupta@chromium.org>
Tested-by: Manoj Gupta <manojgupta@chromium.org>
Reviewed-by: Luis Lozano <llozano@chromium.org>
Reviewed-by: Alex Deymo <deymo@chromium.org>
Reviewed-by: Alex Vakulenko <avakulenko@chromium.org>
| -rw-r--r-- | brillo/http/http_request_unittest.cc | 2 | ||||
| -rw-r--r-- | 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<MockStream> 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. |