Re: [PATCH v2 09/12] fsmonitor: implement filesystem change listener for Linux
- From
Ævar Arnfjörð Bjarmason <avarab@gmail.com>
- Date
- Oct 18, 2022, 11:59 UTC
- Message-ID
- <221018.86zgdtibif.gmgdl@evledraar.gmail.com>
- In-Reply-To
- <4f9c5358475867af75acd865505884f99801ca17.1665783945.git.gitgitgadget@gmail.com>
On Fri, Oct 14 2022, Eric DeCosta via GitGitGadget wrote:
Show 5 quoted lines
> From: Eric DeCosta <edecosta@mathworks.com> > [...] > + 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!