Attention is currently required from: laforge, pespin, keith.

View Change

1 comment:

• File src/input/e1d.c:

  • Patch Set #4, Line 123: if (bfd->list.next && bfd->list.next != LLIST_POISON1)

    No, it doesn't, because osmo_fd_is_registered() can be costly, and in most ocassions you know if the […]

    Indeed osmo_fd_is_registered() is a bit costly due to the list iteration.

    You are also right about the fd >= 0 check. This makes sense. Thanks.

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