Attention is currently required from: laforge, neels.
2 comments:
Patchset:
regarding split files, i kind of like that everything is contained in a single .c file. […]
With thew new comments you added it's "good enough" because one can clearly see which thread executes that function, so we can merge it this way for now.
My point is that, precisely, if you put all the nft thread in a seprate file, it suddenly becomes clear that all that code belongs to that thread, and you don't need to go marking each function specifying who runs it.
Furthermore, it becomes easier to follow interaction of 2 threads by having one file open by the side of the other one.
File src/osmo-hnbgw/nft_kpi.c:
Patch Set #12, Line 694: unsigned long period_us = osmo_tdef_get(hnbgw_T_defs, -34, OSMO_TDEF_US, 1000000);
ah yes this is exactly what i thought
yes, this is simple, usual in code, so I'm not asking something out of this world I think. Feel free to submit as a follow-up patch, but I definetly think this should be merged soon after this patch is merged.
To view, visit change 36539. To unsubscribe, or for help writing mail filters, visit settings.