This is merely a historical archive of years 2008-2021, before the migration to mailman3.
A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/baseband-devel@lists.osmocom.org/.
Vadim Yanitskiy axilirator at gmail.comHi Harald, > I've done some casual review and it looks great to me. Thanks. > I would suggest internal symbols (those not with osmo_gapk_ prefix, > like gapk_log_subsys, LOGPAPK, ...) use an internal header file which > is not installed in the system. The header files installed during > 'make install' should only describe the external API of the library The mentioned logging related symbols are not installed: > noinst_HEADERS = \ > osmocom/gapk/logging.h \ > ... The only thing I am not sure about is 'get_cycles.h' header. At the moment, it is installed together with other headers, because it's a dependency of 'benchmark.h'. But, it seems I just got an idea, how to avoid this dependency - simply use 'unsigned long long' instead of the 'cycles_t', which is imported from 'get_cycles.h'. > the source/sink/coec category name stringification should not > define those strings in every file, but have a value_string > or at least some #defines or the like. Thanks, good idea. > I suggest you simply rebase your chain of commits on current > master and push it into gerrit. But this way, each commit of the whole chain would require +2 and V+1, because 'make check' was implemented at the end of the chain. What if I merge 'fixeria/lib' into a local 'master' and send it for review? I think, this would be simpler. Or would this anyway send the whole chain to review? :) With best regards, Vadim Yanitskiy. -------------- next part -------------- An HTML attachment was scrubbed... URL: <http://lists.osmocom.org/pipermail/baseband-devel/attachments/20180114/ea22f50e/attachment.htm>