On 2/15/2022 10:59 AM, Jeff Hostetler via GitGitGadget wrote: > From: Jeff Hostetler <jeffhost@xxxxxxxxxxxxx> > diff --git a/compat/fsmonitor/fsm-health-win32.c b/compat/fsmonitor/fsm-health-win32.c > index 3c3453369cd..2526ad9194f 100644 > --- a/compat/fsmonitor/fsm-health-win32.c > +++ b/compat/fsmonitor/fsm-health-win32.c > @@ -14,7 +14,10 @@ enum interval_fn_ctx { CTX_INIT = 0, CTX_TERM, CTX_TIMER }; > typedef int (interval_fn)(struct fsmonitor_daemon_state *state, > enum interval_fn_ctx ctx); > > +static interval_fn has_worktree_moved; > + > static interval_fn *table[] = { > + has_worktree_moved, > NULL, /* must be last */ > }; Looking at this now, I think table[] should be defined immediately before fsm_health__loop() so it is easier to see how they interact. It also avoids this static declaration of the function before its implementation. Or, is there a reason it is so high up in the file? Thanks, -Stolee