Attention is currently required from: manawyrm, laforge, tnt.

Patch set 1:Code-Review +1

View Change

1 comment:

• Patchset:

  • Patch Set #1:

    Looks fine, but :
     - Why is that needed ? I don't see it used in subsequent patches (yet) ?
     - I'm wondering if 'len' should be ssize_t just for consitency. (I know it can't be size_t because we use -1 as special case which for 'size' is not an option). But that's mostly cosmetic as we'll never have packets larger than 2 GB 😊

To view, visit change 27578. To unsubscribe, or for help writing mail filters, visit settings.