On Mon, 17 Feb 2025 at 09:55, Thomas Munro <thomas.mu...@gmail.com> wrote: > > Hi, > > Here are some patches that address some of Andres's feedback since the > AIO v2 rebase[1], anticipate out-of-order streams, and make some other > minor improvements. They are independent of the main AIO patch set > and apply to master, hence separate thread.
Hi, great! > 0001-Refactor-read_stream.c-s-circular-arithmetic.patch > > This just replaced open-coded arithmetic with inline functions. They > will be used a lot more in later work, and provide central places to > put assertions that were not checked as uniformly as I would like. Just out of curiosity, should we `Assert(*index + n < stream->queue_size);` in `read_stream_index_advance_n`? -- Best regards, Kirill Reshke