Am 23.11.2022 um 12:42 hat Emanuele Giuseppe Esposito geschrieben: > This macro will be used to mark all coroutine_fn functions. > Right now, it will be used for the newly introduced coroutine_fn, since > we know the callers. > > As a TODO, in the future we might want to add this macro to all > corotuine_fn functions, to be sure that they are only called in
s/corotuine_fn/coroutine_fn/ > coroutines context. > > Signed-off-by: Emanuele Giuseppe Esposito <eespo...@redhat.com> I already asked about other opinions on this in patch 1. These assertions are runtime checks and I don't feel they are the right tool to verify coroutine_fn consistency. Asserting in tricky places makes sense to me, especially as long as we can't rely on static analysis, but adding it everywhere feels over the top to me. Kevin
