On Wed, 9 Oct 2019 16:29:09 +0100
Flavia Musatescu <[email protected]> wrote:
> + if (sendto(pkt_q->sockfd, NULL, 0, MSG_DONTWAIT, NULL, 0) == -1 &&
> + errno != ENOBUFS) {
> + /* Error sending.
> + * When sendto call fails and ENOBUFS error is being set
> + * some of the packets are actually successfully transmitted.
> + * There is no available count of those packets, so in order
> + * to make the statistics more accurate, all of the previously
> + * enqueued packets will be considered successful, even though
> + * this is not entirely correct.
> + */
Agree with the change, but please keep only add a small code comment.
Long comments are a personal pet peeve, it also tends to standout as
"some different developer who had an issue left this comment"