On Thu, May 14, 2020 at 01:39:45PM -0700, Pallavi Kadam wrote:
> This patch fixes bunch of warnings when compiling on Windows
> such as the use of an unsafe string function (strerror),
> [-Wunused-const-variable] in getopt.c and
> [-Wunused-variable], [-Wunused-function] in eal_common_options.c
> 
> v2 changes:
>       Excluded dirent.h file on Windows temporarily.
>       (This file will stay on Windows for later use)
> 
> Signed-off-by: Ranjit Menon <ranjit.me...@intel.com>
> Signed-off-by: Pallavi Kadam <pallavi.ka...@intel.com>
> Tested-by: Pallavi Kadam <pallavi.ka...@intel.com>
> ---
> 
Glad to see those warnings gone!
Tested with clang, FYI it looks like there's only one warning left. We use 
"attr" in the pthread_create call, but in the pthread_create macro definition 
we ignore the attribute parameter.
[13/30] Compiling C object 
lib/76b5a35@@rte_eal@sta/librte_eal_common_eal_common_thread.c.obj.
../lib/librte_eal/common/eal_common_thread.c:180:25: warning: unused parameter 
'attr' [-Wunused-parameter]
                const pthread_attr_t *attr,
                                      ^
1 warning generated.

Otherwise,
Acked-by: Narcisa Vasile <navas...@linux.microsoft.com>

Reply via email to