commit cfc77602c2a80cf8da9b39258fd8d7fc478a12e6
author Ben Clayton <bclayton@google.com> Thu Apr 30 16:48:41 2020 +0100
committer Ben Clayton <bclayton@google.com> Thu Apr 30 19:13:05 2020 +0000
tree 52214caf40945c14eba51edf004896ad09f29433
parent 197e251432d88cd544c23444991f086558cb1cd3

Vulkan/Debug: Fix build with clang-10.

Fix warning treated as error that loop only iterates once.

Bug: b/148401179
Change-Id: I8caf237ede034ba6802a6fbc8167140b27cc13cb
Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/44548
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
Tested-by: Ben Clayton <bclayton@google.com>
Kokoro-Result: kokoro <noreply+kokoro@google.com>

src/Vulkan/Debug/Server.cpp

diff --git a/src/Vulkan/Debug/Server.cpp b/src/Vulkan/Debug/Server.cpp
index 267811d..667caee 100644
--- a/src/Vulkan/Debug/Server.cpp
+++ b/src/Vulkan/Debug/Server.cpp
@@ -307,13 +307,9 @@
 
 			// Workaround for
 			// https://github.com/microsoft/VSDebugAdapterHost/issues/11
-			if(clientIsVisualStudio)
+			if(clientIsVisualStudio && !threads.empty())
 			{
-				for(auto thread : threads)
-				{
-					event.threadId = thread->id.value();
-					break;
-				}
+				event.threadId = threads.front()->id.value();
 			}
 		}