From: degomme Date: Fri, 10 Feb 2017 10:03:12 +0000 (+0100) Subject: some captured params for llambda were unused, triggering new warning in clang 5 X-Git-Tag: v3_15~459 X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/2cbb6e05f75a211ff6241a1d2a281d062ba47bf1?hp=2c96bf50e2ab9d95ffa250c2cc2b19670a7a4414 some captured params for llambda were unused, triggering new warning in clang 5 --- 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);