Phil Sutter <phil@xxxxxx> wrote: > On Wed, Jul 16, 2025 at 11:44:59AM +0200, Florian Westphal wrote: > > Phil Sutter <phil@xxxxxx> wrote: > > > #include <string.h> > > > #include <stdlib.h> > > > +#include <libmnl/libmnl.h> > > > > Why is this include needed? > > Because of: > > | In file included from udata.c:9: > | ../include/utils.h:88:40: warning: 'struct nlattr' declared inside parameter list will not be visible outside of this definition or declaration > | 88 | const char *mnl_attr_get_ifname(struct nlattr *attr); > | | ^~~~~~ That struct is defined in linux/netlink.h. But you can add a foward declaration for this, i.e.: struct nlattr; As the layout isn't needed for the function declaration.
