Subject: Re: [PATCH] fsmonitor-watchman: fix variable reference and remove redundant code

From: Paul Tarjan <[email-masked]>
Date: Sat, 28 Feb 2026 10:33:14 -0700

Koji Nakamaru <[email-masked]> writes:

> Although I don't have much experience with the Watchman backend, the
> fixes look correct to me.
>
> One suggestion: it seems we can make the code even cleaner by removing
> $retry and its associated logic, as they appear to be no longer
> necessary after these changes.

Good point. $retry only existed to prevent the infinite recursion
from the recursive launch_watchman() call, so now that we've
removed that call it's dead code. Cleaned it up in v2.

Thanks for the review,
Paul