"Eric DeCosta via GitGitGadget" <gitgitgadget@xxxxxxxxx> writes: > +const char *fsmonitor_ipc__get_path(void) > +{ > +#ifdef WIN32 > + return fsmonitor_ipc__get_default_path(); > +#else Hmph. As compat/fsmonitor/ directory already sets up a good way to have platform-specific implementation of a common API function, I think this patch goes in a wrong direction by fighting it. Shouldn't the rest of the function we see here be made to another implementation of fsmonitor_ipc__get_default_path() that is specific to platforms, i.e. in compat/fsmonitor/$somenewfile.c, that is used for !WIN32 case?