On 10/19/21 01:25, Andreas Schwab wrote: > On Okt 19 2021, Paul Eggert wrote: > >> + ch = (mctx.input.valid_len <= offset > > This is backwards. It's correct as-is, so that comment is merely about style. I revamped the patch to turn the comparison around; see attached. Let's not have our longstanding style disagreement distract us from the fix.