commit 9832a51c8b5d6ea3e980bf0251523e9ebf301b68
from: Stefan Sperling <stsp@stsp.name>
via: Thomas Adam <thomas@xteddy.org>
date: Tue Jun 25 21:23:16 2024 UTC

Remove unnecessary sched_yield(2).

Re-applies commit 6b8a2b8fcd99c4365b1aa9513c0f0149beac2491 which was
reverted in commit 82954512f323c8a2d4f89d51be1e6b0f707b6c3a because
at the time the change broke the 'search' feature.
Quick testing suggests search is now working fine, so let's try again.

commit - fe0e729042492b0efa2dd74dc82fca75e5383e22
commit + 9832a51c8b5d6ea3e980bf0251523e9ebf301b68
blob - 23d3862bd60b306cd65d724e25b63a2ceb8ce659
blob + 094e8372a6a869db30b8b7943645cdb78c02103c
--- tog/tog.c
+++ tog/tog.c
@@ -42,7 +42,6 @@
 #include <pthread.h>
 #include <libgen.h>
 #include <regex.h>
-#include <sched.h>
 
 #include "got_version.h"
 #include "got_error.h"
@@ -1721,15 +1720,6 @@ view_input(struct tog_view **new, int *done, struct to
 	}
 
 	if (view->searching && !view->search_next_done) {
-		errcode = pthread_mutex_unlock(&tog_mutex);
-		if (errcode)
-			return got_error_set_errno(errcode,
-			    "pthread_mutex_unlock");
-		sched_yield();
-		errcode = pthread_mutex_lock(&tog_mutex);
-		if (errcode)
-			return got_error_set_errno(errcode,
-			    "pthread_mutex_lock");
 		view->search_next(view);
 		return NULL;
 	}