[LTP] [PATCH] pthread_cond_init/s-c.c: Add the lose ifndef line
Yang Xu
xuyang2018.jy@cn.fujitsu.com
Tue Mar 17 03:08:02 CET 2020
Hi Zou
Even this case does't compile or run on ltp default envrionment, I think
it is a framework problem about ltp openposix. This fix is obviously
correct and simple.
Reviewed-by: Yang Xu <xuyang2018.jy@cn.fujitsu.com>
Someone who merging this patch can add fixes tag.
Fixes: 310c9eb6e ("Remove explicit #define's for `_POSIX_C_SOURCE` and
`_XOPEN_SOURCE`")
Best Regards
Yang Xu
> Hi Zou
>
> It was introduced by commit 310c9eb6e6("Remove explicit #define's for
> `_POSIX_C_SOURCE` and `_XOPEN_SOURCE`").
>
> IMO, we can remove this WITHOUT_XOPEN macro directly because we have
> setted -D_POSIX_C_SOURCE=200809L -D_XOPEN_SOURCE=700 in
> ltp/testcases/open_posix_testsuite/CFLAGS.
>
> Best Regards
> Yang Xu
>> pthread_cond_init/s-c.c testcase lose a #ifndef line, fix it
>>
>> Reported-by: Zou Wei <zou_wei@huawei.com>
>> Signed-off-by: Zou Wei <zou_wei@huawei.com>
>> ---
>>
>> testcases/open_posix_testsuite/stress/threads/pthread_cond_init/s-c.c
>> | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git
>> a/testcases/open_posix_testsuite/stress/threads/pthread_cond_init/s-c.c b/testcases/open_posix_testsuite/stress/threads/pthread_cond_init/s-c.c
>>
>> index 6a5c70a..8d18484 100644
>> ---
>> a/testcases/open_posix_testsuite/stress/threads/pthread_cond_init/s-c.c
>> +++
>> b/testcases/open_posix_testsuite/stress/threads/pthread_cond_init/s-c.c
>> @@ -80,6 +80,7 @@
>> /*********************************** Test case
>> *****************************************/
>>
>> /********************************************************************************************/
>>
>> +#ifndef WITHOUT_XOPEN
>> typedef struct _teststruct {
>> pthread_cond_t cnd[10 * SCALABILITY_FACTOR];
>> pthread_condattr_t ca[4];
>> --
>> 2.6.2
>>
>>
>
>
>
More information about the ltp
mailing list