On Wed, Oct 14, 2026 at 08:31:22AM +0200, Antonin Houska wrote:
> Although there's no compiler warning, and probably no correctness issue, I
> think the intended coding was as I propose in the diff attached.
Indeed. That's harmless still inconsistent as this *buffer_invalid
points to error_count in buffer_readv_encode_error, which is assumed
to be a uint8 counter.
I am wondering if this should be backpatched down to v18, actually, at
least to keep the branches consistently in check, or just as a matter
of style.
--
Michael