Attention is currently required from: arehbein, laforge, pespin.
1 comment:
Patchset:
It can know based on the OSMO_IO_FD_MODE configured during osmo_iofd_setup(). […]
I disagree that we should put the burden of this onto the user. It's an easy enough mistake to make.
This is complicated by the fact that the only way to set (either initially or later) the callback function is by passing in a struct osmo_iofd_ops *. This means that with the union there is *no* way at all to detect this.
```
osmo_iofd_setup(iofd, ..., mode, &ops); // Checking ops callbacks according to mode doesn't work
osmo_iofd_set_ioops(iofd, &ops); // Same as above, we have iofd->mode, but ops is ambiguous
```
Maybe you're thinking of osmo_stream_* where we have *_set_read_cb() etc. functions?
To view, visit change 35079. To unsubscribe, or for help writing mail filters, visit settings.