3 Replies to “Yet another network optimization”

  1. There are missing brackets or wrong indentation. Most likely wrong indentation.

    + if (ai != NULL)
    + error = udp_preattach(so, 0 /* don’t care */, ai);
    + if (error)
    + goto out;
    + else {
    + /* Post attach; do nothing */
    + }

  2. hmm, s4 is correct.
    I think it’s missing brackets after != NULL) myself.

  3. Oh :)

    Pretty accidental. But the effect is same :), it currently read:
    int error = 0;

    if (ai != NULL)
    error = udp_preattach();

    if (error) {
    goto out;
    } else {
    /* comment */
    }

    I am fixing it now :)

Comments are closed.