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