On Wed, Jul 01, 2026 at 08:00:38PM +0530, Ayush Tiwari wrote:
> v2 attached sets NLocBuffer = nbufs before the initialization loop, so the
> loop can keep using GetLocalBufferDescriptor(), and drops the now-redundant
> assignment at the end.
Catching up with the latest activity of this thread now that v20 has
opened up. That's kind of nice.
I have been also reminded about BufferIsValid(), where we already have
similar assertions. As a whole, that seems acceptable to me, and
using unsigned still does not make much sense here.
Any thoughts or objections regarding a HEAD-only change?
--
Michael
