reftable/stack: stop using sleep_millisec()

pks-t · gitster · commit 10f2935c7f20 · 2025-02-18T10:55:38.000-08:00
Refactor our use of `sleep_millisec()` by open-coding it with poll(3p),
which is the current implementation of this function. Ideally, we'd use
a more direct way to sleep, but there is no equivalent to sleep(3p) that
would accept milliseconds as input.

Signed-off-by: Patrick Steinhardt &lt;ps@pks.im&gt;
Signed-off-by: Junio C Hamano &lt;gitster@pobox.com&gt;
diff --git a/reftable/stack.c b/reftable/stack.c
@@ -524,7 +524,7 @@ static int reftable_stack_reload_maybe_reuse(struct reftable_stack *st,
 		fd = -1;
 
 		delay = delay + (delay * reftable_rand()) / UINT32_MAX + 1;
-		sleep_millisec(delay);
+		poll(NULL, 0, delay);
 	}
 
 out:

Original file line number	Diff line number	Diff line change
`@@ -524,7 +524,7 @@ static int reftable_stack_reload_maybe_reuse(struct reftable_stack *st,`
`524`	`524`	`fd = -1;`
`525`	`525`
`526`	`526`	`delay = delay + (delay * reftable_rand()) / UINT32_MAX + 1;`
`527`		`- sleep_millisec(delay);`
	`527`	`+ poll(NULL, 0, delay);`
`528`	`528`	`}`
`529`	`529`
`530`	`530`	`out:`