On Fri, Dec 3, 2021 at 10:00 PM Ævar Arnfjörð Bjarmason <avarab@xxxxxxxxx> wrote: > > > On Fri, Dec 03 2021, Han Xin wrote: > > > + unsigned long bufsize = dry_run ? 4096 : size; > > + void *buf = xmallocz(bufsize); > > It's probably nothing, but in your CL you note that you changed another > hardcoding from 4k to 8k, should this one still be 4k? > > It's probably fine, just wondering... Yes, I think this is an omission from my work.