On Fri, Nov 08, 2019 at 10:00:35AM -0800, Stephen Hemminger wrote: > The #ifdef to conditionally include <sys/socket.h> on BSD > is unnecessary. It is harmless to include the header on other > OS's. An extra include is better than an #ifdef. > > Signed-off-by: Stephen Hemminger <step...@networkplumber.org> > --- > app/test-pmd/cmdline.c | 2 -- > app/test/test_cmdline_ipaddr.c | 2 -- > lib/librte_cmdline/cmdline_parse_ipaddr.c | 2 -- > 3 files changed, 6 deletions(-) > Seems reasonable
Acked-by: Bruce Richardson <bruce.richard...@intel.com>
