On Fri, Oct 14 2022, Eric DeCosta via GitGitGadget wrote: > From: Eric DeCosta <edecosta@xxxxxxxxxxxxx> > [...] > + strbuf_add(&path, w->dir, strlen(w->dir)); > + strbuf_addch(&path, '/'); > + strbuf_add(&path, event->name, strlen(event->name)); Don't do strbuf_add(&buf, x, strlen(x), just use strbuf_addstr(&buf, x) instead. The same goes for a couple of existing occurances that hit "master" already in the just-merged fsmonitor topic, but in this case we can change it in-flight still. Thanks!