Save a call to stopThreadedIOIfNeeded() for the base case.

Probably no performance changes, but the code should be trivial to
read as in "No threading? Use the normal function and return".
This commit is contained in:
antirez 2020-05-01 18:45:51 +02:00
parent 1367cb7b6d
commit ef0c2a9268

View File

@ -2997,9 +2997,9 @@ int handleClientsWithPendingWritesUsingThreads(void) {
int processed = listLength(server.clients_pending_write); int processed = listLength(server.clients_pending_write);
if (processed == 0) return 0; /* Return ASAP if there are no clients. */ if (processed == 0) return 0; /* Return ASAP if there are no clients. */
/* If we have just a few clients to serve, don't use I/O threads, but the /* If I/O threads are disabled or we have few clients to serve, don't
* boring synchronous code. */ * use I/O threads, but thejboring synchronous code. */
if (stopThreadedIOIfNeeded()) { if (server.io_threads_num == 1 || stopThreadedIOIfNeeded()) {
return handleClientsWithPendingWrites(); return handleClientsWithPendingWrites();
} }