From 2cbb6e05f75a211ff6241a1d2a281d062ba47bf1 Mon Sep 17 00:00:00 2001 From: degomme Date: Fri, 10 Feb 2017 11:03:12 +0100 Subject: [PATCH] some captured params for llambda were unused, triggering new warning in clang 5 --- include/simgrid/simix/blocking_simcall.hpp | 2 +- src/msg/msg_vm.cpp | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/include/simgrid/simix/blocking_simcall.hpp b/include/simgrid/simix/blocking_simcall.hpp index 3a41904109..6341521a42 100644 --- a/include/simgrid/simix/blocking_simcall.hpp +++ b/include/simgrid/simix/blocking_simcall.hpp @@ -94,7 +94,7 @@ public: simcall_run_blocking([this, &result, self]{ try { // When the kernel future is ready... - this->future_.then_([this, &result, self](simgrid::kernel::Future value) { + this->future_.then_([&result, self](simgrid::kernel::Future value) { // ... wake up the process with the result of the kernel future. simgrid::xbt::setPromise(result, value); simgrid::simix::unblock(self); diff --git a/src/msg/msg_vm.cpp b/src/msg/msg_vm.cpp index 64e13a69ff..6d1d5c320c 100644 --- a/src/msg/msg_vm.cpp +++ b/src/msg/msg_vm.cpp @@ -290,11 +290,10 @@ static int migration_rx_fun(int argc, char *argv[]) // This is clearly ugly but I (Adrien) need more time to do something cleaner (actually we should copy the whole ms // structure at the beginning and free it at the end of each function) simgrid::s4u::VirtualMachine* vm = static_cast(ms->vm); - msg_host_t src_pm = ms->src_pm; msg_host_t dst_pm = ms->dst_pm; // Make sure that we cannot get interrupted between the migrate and the resume to not end in an inconsistent state - simgrid::simix::kernelImmediate([vm, src_pm, dst_pm]() { + simgrid::simix::kernelImmediate([vm, dst_pm]() { /* Update the vm location */ /* precopy migration makes the VM temporally paused */ xbt_assert(vm->pimpl_vm_->getState() == SURF_VM_STATE_SUSPENDED); -- 2.20.1