On Thu, May 8, 2025 at 11:38 PM Heikki Linnakangas <hlinn...@iki.fi> wrote: > It didn't occur to me that you could write it simply as 'msgLength - 4'. > That depends on knowing that the preceding fields are exactly 4 bytes > long, but that's clear enough if we just add a comment on that, see > attached.
Sorry for the conference delay; this looks fine to me. One of the side effects of the uint8 change is that the client now accepts cancel keys up to roughly 30kb. Is that intended? --Jacob
