<div dir="ltr"><div><div><div><div>Hi,<br><br>> written manually?<br><br></div>Yes. Just look at content of this header file.<br><br>> We tend to put all headers used from several locations in<br>
> openbsc/openbsc/include/openbs<wbr>c/ (as noinst_HEADERS in the Makefile.am).<br>
> To #include, you would use something like #include <openbsc/conv.h>.<br><br></div>Yeah, I tried to put one into include/osmocom/tests/, but Harald and Sylvain<br></div>voted against this approach. So I decided to go this way.<br></div><div><div><div><br></div></div></div></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature" data-smartmail="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 class="gmail_quote">2017-05-02 17:52 GMT+07:00 Neels Hofmeyr <span dir="ltr"><<a href="mailto:nhofmeyr@sysmocom.de" target="_blank">nhofmeyr@sysmocom.de</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Mon, May 01, 2017 at 03:41:03AM +0700, Vadim Yanitskiy wrote:<br>
> > We generate "conv.h" and put it into the source directory?<br>
><br>
> Actually, no. We aren't generate the "conv.h".<br>
> It's written manually and contains the conv_test_vector<br>
> structure definition and the do_check() function definition.<br>
<br>
</span>written manually?  You mean committed in gerrit, right?<br>
<span class=""><br>
> One was introduced by: <a href="https://gerrit.osmocom.org/#/c/1627/" rel="noreferrer" target="_blank">https://gerrit.osmocom.org/#/<wbr>c/1627/</a><br>
> so now the same test logic is used in two different tests<br>
> (both conv and conv_gsm0503) without code duplication.<br>
<br>
</span>We tend to put all headers used from several locations in<br>
openbsc/openbsc/include/<wbr>openbsc/ (as noinst_HEADERS in the Makefile.am).<br>
To #include, you would use something like #include <openbsc/conv.h>.<br>
<span class=""><br>
> Don't you confuse it with the "gsm0503.h", which is exactly<br>
> generated by the "utils/conv_gen.py"?<br>
<br>
</span>Loosely related: some time back I modified some generation code to put<br>
generated C files in the builddir, adding -I$(builddir) to be able to find<br>
them from srcdir... I think it was this one I fixed.<br>
<span class="HOEnZb"><font color="#888888"><br>
~N<br>
<br>
</font></span></blockquote></div><br></div>