Hi Vadim,
just some feedback "from the side":
On Sun, Jan 14, 2018 at 02:19:46AM +0600, Vadim Yanitskiy wrote:
I've done some casual review and it looks great to me. There are
some minor issues:
* 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 source/sink/coec category name stringification at
http://git.osmocom.org/gapk/commit/?h=fixeria/lib&id=459791c488c6b66a5c…
should not define those strings in every file, but have a value_string
or at least some #defines or the like.
This is just my point of view, detailed review of course depends on Sylvain.
I suggest you simply rebase your chain of commits on current master and push
it into gerrit. That's what we have it for, and as indicated in
http://lists.osmocom.org/pipermail/openbsc/2017-December/011523.html
gapk has been introduced to gerrit.
Regards,
Harald
--
- Harald Welte <laforge(a)gnumonks.org>
http://laforge.gnumonks.org/
============================================================================
"Privacy in residential applications is a desirable marketing option."
(ETSI EN 300 175-7 Ch. A6)