On Wed, Nov 13, 2019 at 09:23:02AM -0800, Gregory Rose wrote:
> 
> On 11/5/2019 5:03 PM, Gregory Rose wrote:
> > 
> > On 11/5/2019 3:14 PM, Ben Pfaff wrote:
> > > On Tue, Nov 05, 2019 at 02:14:24PM -0800, Greg Rose wrote:
> > > > The missing inline keyword before the definition of the
> > > > rpl_nf_ct_tmpl_free() function causes spurious warnings about the
> > > > function not being used on some older kernels.  Add the keyword
> > > > to suppress the warning.

...

> This is a benign issue for now but if anyone ever tried to use
> rpl_nf_ct_tmpl_free in two different modules they'd
> get a linker error.  That probably won't happen but it's still not correct.

Even without "inline", it's still declared "static", which means that it
can't duplicate symbols defined in other translation units.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to