Re: [PATCH 2/4] reftable/merged: fix zero-sized allocation when there are no readers
- From
Patrick Steinhardt <ps@pks.im>
- Date
- Dec 21, 2024, 14:18 UTC
- Message-ID
- <Z2bOOn4Oad9b9Ycr@pks.im>
- In-Reply-To
- <b4ba502a-b689-4736-9712-443a137b5d2c@app.fastmail.com>
On Sat, Dec 21, 2024 at 01:40:32PM +0100, Kristoffer Haugsbakk wrote:
Show 6 quoted lines
> Hi > > On Sat, Dec 21, 2024, at 12:50, Patrick Steinhardt wrote: > > It was reported [1c > > A Neo user, I see.
There are dozens of us!
> s/c/]/
Oh, I even saw that typo, but obviously forgot to fix it.
Show 8 quoted lines
> > [snip] > > the result. Now that we do we notice and thus return an out-of-memory > > error to the caller. > > s/Now that we do we/Now we do notice/ > > Repeat of “we”. And “that” can’t be be used since it has nothing to > connect to.
Indeed. Will wait a bit to hear back from Randall before sending another version. Thanks!
Patrick