On Tue, 3 Oct 2017 13:14:59 -0700 Brandon Williams <bmwill@xxxxxxxxxx> wrote: > +void packet_write(const int fd_out, const char *buf, size_t size) No need for "const" in "const int fd_out", I think. Same comment for the header file. > +{ > + if (packet_write_gently(fd_out, buf, size)) > + die_errno("packet write failed"); > +}