diff mbox

validation: init: include stdlib.h for abort()

Message ID 1423145747-32463-1-git-send-email-taras.kondratiuk@linaro.org
State Accepted
Commit 1e6fd1a68c0fb08c112cf5717f0ff19335b1db80
Headers show

Commit Message

Taras Kondratiuk Feb. 5, 2015, 2:15 p.m. UTC
Signed-off-by: Taras Kondratiuk <taras.kondratiuk@linaro.org>
---
 test/validation/odp_init_abort.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Maxim Uvarov Feb. 8, 2015, 9:20 p.m. UTC | #1
Taras, do you see compilation error without stdlib.h?

Maxim.

On 02/05/2015 05:15 PM, Taras Kondratiuk wrote:
> Signed-off-by: Taras Kondratiuk <taras.kondratiuk@linaro.org>
> ---
>   test/validation/odp_init_abort.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/test/validation/odp_init_abort.c b/test/validation/odp_init_abort.c
> index c291146..ceb82b5 100644
> --- a/test/validation/odp_init_abort.c
> +++ b/test/validation/odp_init_abort.c
> @@ -5,6 +5,7 @@
>    */
>   
>   #include <stdarg.h>
> +#include <stdlib.h>
>   #include <odp.h>
>   #include <CUnit/Basic.h>
>
Taras Kondratiuk Feb. 24, 2015, 2:12 p.m. UTC | #2
On 02/08/2015 11:20 PM, Maxim Uvarov wrote:
> Taras, do you see compilation error without stdlib.h?

Sorry I've missed your reply.

Yes I see it in linux-keystone implementation.
You don't see it in linux-generic just because stdlib.h is included via
some ODP header. Tests should not rely on this.
Maxim Uvarov Feb. 24, 2015, 2:36 p.m. UTC | #3
Merged!

Maxim.

On 02/24/2015 05:12 PM, Taras Kondratiuk wrote:
> On 02/08/2015 11:20 PM, Maxim Uvarov wrote:
>> Taras, do you see compilation error without stdlib.h?
> Sorry I've missed your reply.
>
> Yes I see it in linux-keystone implementation.
> You don't see it in linux-generic just because stdlib.h is included via
> some ODP header. Tests should not rely on this.
>
diff mbox

Patch

diff --git a/test/validation/odp_init_abort.c b/test/validation/odp_init_abort.c
index c291146..ceb82b5 100644
--- a/test/validation/odp_init_abort.c
+++ b/test/validation/odp_init_abort.c
@@ -5,6 +5,7 @@ 
  */
 
 #include <stdarg.h>
+#include <stdlib.h>
 #include <odp.h>
 #include <CUnit/Basic.h>