Attention is currently required from: osmith, laforge.
5 comments:
File src/host/trxcon/src/l1ctl.c:
Patch Set #3, Line 467: /* FIXME: we cannot know for sure if the given mode was actually applied.
can this be fixed? the comment sounds like it can't but there's a FIXME infront
No, it cannot be fixed. This is a limitation of the osmo_fsm API: you cannot indicate a negative outcome from the .action callback. I we were in C++ land, I would simply raise an exception, but this is C. Adding this FIXME to not forget about this problem. Maybe I'll find a solution later.
File src/host/trxcon/src/trxcon_fsm.c:
Patch Set #3, Line 52: /* TODO: osmo_fsm_inst_term(fi, OSMO_FSM_TERM_ERROR, NULL); */
kept on purpose?
Yes. This is going to be fixed in a follow up patch, not here.
Patch Set #3, Line 127: /* TODO: timeout */
kept on purpose?
Yes, this is a potential improvement, so that's why TODO.
missing \n
Nice catch! Thanks!
Patch Set #3, Line 280: /* TODO: set proper .snr */
kept on purpose? same below
This TODO existed before this commit, so I'll keep it.
To view, visit change 28809. To unsubscribe, or for help writing mail filters, visit settings.