Minecon724/dncurrency
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

Add unnecessary lambda capture to please VS2017 (#1988)

Browse source
This commit is contained in:
cryptocode 2019-05-16 10:19:52 +02:00 • committed by GitHub
commit a4cc47cd87
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
nano/core_test/socket.cpp
View file

@ -87,7 +87,9 @@ TEST (socket, concurrent_writes)
auto client (clients[0]);
for (int i = 0; i < client_count; i++)
{
std::thread runner ([&client]() {
// Note: this gives a warning on most compilers because message_count is constexpr and a
// capture isn't needed. However, removing it fails to compile on VS2017 due to a known compiler bug.
std::thread runner ([&client, &message_count]() {
for (int i = 0; i < message_count; i++)
{
auto buff (std::make_shared<std::vector<uint8_t>> ());