On 3/30/23 16:16, [email protected] wrote: > From: Lin Huang <[email protected]> > > Declaration of 'struct conn' will not be visible outside of this function. > Declaration of 'struct conntrack' will not be visible outside of this > function. > Declaration of 'struct timeout_policy' will not be visible outside of this > function. > > Signed-off-by: Lin Huang <[email protected]> > --- > lib/conntrack-tp.h | 5 +++++ > 1 file changed, 5 insertions(+) >
I can't reproduce the warning on a current master, but the change looks correct to me. Would be nice to add some empty lines to the file though to visually split macros, structs and functions. > diff --git a/lib/conntrack-tp.h b/lib/conntrack-tp.h > index 4d411d19f..2227f6fa5 100644 > --- a/lib/conntrack-tp.h > +++ b/lib/conntrack-tp.h > @@ -17,8 +17,13 @@ > #ifndef CONNTRACK_TP_H > #define CONNTRACK_TP_H 1 > > +#include <stdint.h> > + > #define CT_DPIF_NETDEV_TP_MIN 30 An empty line here. > enum ct_timeout; > +struct conn; > +struct conntrack; > +struct timeout_policy; And here. > void timeout_policy_init(struct conntrack *ct); > int timeout_policy_update(struct conntrack *ct, struct timeout_policy *tp); > int timeout_policy_delete(struct conntrack *ct, uint32_t tp_id); If you can add these empty lines and send v2, that would be great. Thanks! Best regards, Ilya Maximets. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
