<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small">Hi Xu,</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Jul 30, 2020 at 3:00 PM Yang Xu <<a href="mailto:xuyang2018.jy@cn.fujitsu.com">xuyang2018.jy@cn.fujitsu.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Cyril<br>
<br>
<br>
> Hi!<br>
>> Signed-off-by: Yang Xu <<a href="mailto:xuyang2018.jy@cn.fujitsu.com" target="_blank">xuyang2018.jy@cn.fujitsu.com</a>><br>
>> ---<br>
>>   include/libnewipc.h           |  28 ++++------<br>
>>   libs/libltpnewipc/libnewipc.c | 101 ++++++++++++++++++++++++++++++++--<br>
>>   2 files changed, 109 insertions(+), 20 deletions(-)<br>
> <br>
> Just FYI we do not have to put all the code into a single *.c file, we<br>
> can have as many as we want in the library directory...<br>
I remember you have some comments on my previous patach as below:<br>
------------------------------<br>
So unless we plan to use it from different tests it does not make much<br>
sense to export it to the top level include and libs.<br>
<br>
Or alternatively we do have libltpnewipc we can pust put the<br>
libnewmsgctl.c to that directory and with that we will have a single<br>
System V IPC helper library.<br>
------------------------------<br>
<br>
So I merge libnewmsgctl.c into libnewipc.c.<br>
<br>
ps: IMO, libnewipc has only three functions(getipckey,get_used_queues, <br>
probe_free_addr) ,It's hard to call it a LIBS library. Can we put it <br>
into lib directory and named it as tst_ipc.c?<br>
@LI, What do you think about this?<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I have no objection to this.</div></div><div><br></div><div><div class="gmail_default" style="font-size:small">The reason to move it to LIBS is to support both new and old test cases in same directory, so if you're going to move back into the general LTP lib, we have to convert all the old-tests into new API. Then it will be fine.</div></div><div><br></div></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>