@samitouri / QOSamiQemu / commits / 4f6c5f8fe1

ui/vnc-jobs: drop redundant (and needless) qemu_thread_get_self()

The call is unnecessary, since "thread" is already set at creation time. Furthermore, the "thread" field is mostly useless as the thread is created DETACHED and isn't used for anything but perhaps debugging. Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Signed-off-by: Marc-André Lureau <marcandre.lureau@redhat.com>

Marc-André Lureau committed Mar 13, 2026 at 23:48 UTC 4f6c5f8fe1343dd5bca1965e18af709d6fb77fa4
1 file changed -2
ui/vnc-jobs.c
-2
@@ -354,8 +354,6 @@ static void *vnc_worker_thread(void *arg)
354 {
355 VncJobQueue *queue = arg;
356
357 - qemu_thread_get_self(&queue->thread);
358 -
357 while (!vnc_worker_thread_loop(queue)) ;
358 g_assert_not_reached();
359 return NULL;