Acked-by: Alex Wang <[email protected]>
On Wed, Jun 25, 2014 at 1:44 PM, Ben Pfaff <[email protected]> wrote: > I thought this was in here already, but I guess not. > > Signed-off-by: Ben Pfaff <[email protected]> > --- > CodingStyle | 5 +++++ > 1 file changed, 5 insertions(+) > > diff --git a/CodingStyle b/CodingStyle > index 0ba0366..628f21f 100644 > --- a/CodingStyle > +++ b/CodingStyle > @@ -162,6 +162,11 @@ such a function (including the C standard library > function free()) > should omit a null-pointer check. We find that this usually makes > code easier to read. > > +Functions in .c files should not normally be marked "inline", because > +it does not usually help code generation and it does suppress > +compilers warnings about unused functions. (Functions defined in .h > +usually should be marked inline.) > + > > FUNCTION PROTOTYPES > > -- > 1.7.10.4 > > _______________________________________________ > dev mailing list > [email protected] > http://openvswitch.org/mailman/listinfo/dev > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
