On Thu, Jan 31, 2013 at 07:49:39PM +0900, YAMAMOTO Takashi wrote:
> From: YAMAMOTO Takashi <y...@mwd.biglobe.ne.jp>
> 
> Signed-off-by: YAMAMOTO Takashi <yamam...@valinux.co.jp>

I see many uses of strcasecmp() in the tree:

    blp@blp:~/nicira/ovs(0)$ git --no-pager grep -l strcasecmp
    lib/bundle.c
    lib/cfm.c
    lib/meta-flow.c
    lib/multipath.c
    lib/odp-util.c
    lib/ofp-parse.c
    lib/ofp-util.c
    lib/vlog.c
    utilities/ovs-ofctl.c
    utilities/ovs-vsctl.c
    vswitchd/bridge.c

but only one use of strings.h:

    lib/dpif-linux.c

I am surprised that your patch only adds one #include.  Why is that?

> +#include <strings.h> /* strcasecmp */

The comment isn't necessary here.

Thanks,

Ben.
_______________________________________________
dev mailing list
dev@openvswitch.org
http://openvswitch.org/mailman/listinfo/dev

Reply via email to