<div dir="ltr">Hi Harald,<br><br>> I've done some casual review and it looks great to me.<br><br>Thanks.<br><br>> I would suggest internal symbols (those not with osmo_gapk_ prefix,<br>> like gapk_log_subsys, LOGPAPK, ...) use an internal header file which<br>> is not installed in the system. The header files installed during<br>> 'make install' should only describe the external API of the library<br><br>The mentioned logging related symbols are not installed:<br><br>> noinst_HEADERS = \<br>>     osmocom/gapk/logging.h \<br>>     ...<br><br>The only thing I am not sure about is 'get_cycles.h' header. At the<br>moment, it is installed together with other headers, because it's<br>a dependency of 'benchmark.h'. But, it seems I just got an idea,<br>how to avoid this dependency - simply use 'unsigned long long'<br>instead of the 'cycles_t', which is imported from 'get_cycles.h'.<br><br>> the source/sink/coec category name stringification should not<br>> define those strings in every file, but have a value_string<br>> or at least some #defines or the like.<br><br>Thanks, good idea.<br><br>> I suggest you simply rebase your chain of commits on current<br>> master and push it into gerrit.<br><br>But this way, each commit of the whole chain would require +2<br>and V+1, because 'make check' was implemented at the end of<br>the chain.<br><br>What if I merge 'fixeria/lib' into a local 'master' and send<br>it for review? I think, this would be simpler. Or would this<br>anyway send the whole chain to review? :)<br><br><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>With best regards,<br></div><div>Vadim Yanitskiy.<br></div></div></div></div></div></div>
<br></div></div>