Thanks, applied.

On Thu, Jul 18, 2013 at 03:40:21PM -0700, Justin Pettit wrote:
> Acked-by: Justin Pettit <jpet...@nicira.com>
> 
> 
> 
> On Jul 18, 2013, at 3:35 PM, Ben Pfaff <b...@nicira.com> wrote:
> 
> > There is no reason to mark this function as inline.  Let the compiler do
> > its job.
> > 
> > Signed-off-by: Ben Pfaff <b...@nicira.com>
> > ---
> > lib/ofp-util.c |    2 +-
> > 1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/lib/ofp-util.c b/lib/ofp-util.c
> > index 8921592..bc85797 100644
> > --- a/lib/ofp-util.c
> > +++ b/lib/ofp-util.c
> > @@ -1372,7 +1372,7 @@ ofputil_decode_hello(const struct ofp_header *oh, 
> > uint32_t *allowed_versions)
> > 
> > /* Returns true if 'allowed_versions' needs to be accompanied by a version
> >  * bitmap to be correctly expressed in an OFPT_HELLO message. */
> > -static inline bool
> > +static bool
> > should_send_version_bitmap(uint32_t allowed_versions)
> > {
> >     return !is_pow2((allowed_versions >> 1) + 1);
> > -- 
> > 1.7.2.5
> > 
> > _______________________________________________
> > dev mailing list
> > dev@openvswitch.org
> > http://openvswitch.org/mailman/listinfo/dev
> 
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to