I was not able to apply this to check.

On 6 March 2015 at 11:26, Bill Fischofer <[email protected]> wrote:

>
>
> On Wed, Feb 25, 2015 at 3:30 AM, Taras Kondratiuk <
> [email protected]> wrote:
>
>> stdarg.h is not needed in odp_debug_internal.h, but instead should be
>> included in odp_weak.c.
>>
>> The change is needed to be able to reuse odp_weak.c from other
>> platforms.
>>
>> Signed-off-by: Taras Kondratiuk <[email protected]>
>>
>
> Reviewed-by: Bill Fischofer <[email protected]>
>
>
>> ---
>>  platform/linux-generic/include/odp_debug_internal.h | 1 -
>>  platform/linux-generic/odp_weak.c                   | 2 ++
>>  2 files changed, 2 insertions(+), 1 deletion(-)
>>
>> diff --git a/platform/linux-generic/include/odp_debug_internal.h
>> b/platform/linux-generic/include/odp_debug_internal.h
>> index ff43ffc..d9ac812 100644
>> --- a/platform/linux-generic/include/odp_debug_internal.h
>> +++ b/platform/linux-generic/include/odp_debug_internal.h
>> @@ -18,7 +18,6 @@
>>
>>  #include <stdio.h>
>>  #include <stdlib.h>
>> -#include <stdarg.h>
>>  #include <odp/debug.h>
>>  #include <odp_internal.h>
>>  #ifdef __cplusplus
>> diff --git a/platform/linux-generic/odp_weak.c
>> b/platform/linux-generic/odp_weak.c
>> index 9dc3a4a..9641f45 100644
>> --- a/platform/linux-generic/odp_weak.c
>> +++ b/platform/linux-generic/odp_weak.c
>> @@ -9,6 +9,8 @@
>>  #include <odp_debug_internal.h>
>>  #include <odp/hints.h>
>>
>> +#include <stdarg.h>
>> +
>>  ODP_WEAK_SYMBOL int odp_override_log(odp_log_level_e level ODP_UNUSED,
>>                                      const char *fmt, ...)
>>  {
>> --
>> 1.9.1
>>
>>
>> _______________________________________________
>> lng-odp mailing list
>> [email protected]
>> http://lists.linaro.org/mailman/listinfo/lng-odp
>>
>
>
> _______________________________________________
> lng-odp mailing list
> [email protected]
> http://lists.linaro.org/mailman/listinfo/lng-odp
>
>


-- 
Mike Holmes
Technical Manager - Linaro Networking Group
Linaro.org <http://www.linaro.org/> *│ *Open source software for ARM SoCs
_______________________________________________
lng-odp mailing list
[email protected]
http://lists.linaro.org/mailman/listinfo/lng-odp

Reply via email to