Bleep bloop.  Greetings Roi Dayan, I am a robot and I have tried out your patch.
Thanks for your contribution.

I encountered some error that I wasn't expecting.  See the details below.


checkpatch:
WARNING: The subject summary should end with a dot.
Subject: dpif-plugin: Add pluggable dpif api
WARNING: Comment with 'xxx' marker
#66 FILE: lib/dpif-plugin.c:17:
/* XXX: Use libtool ltdl instead of dlopen ? */

ERROR: Inappropriate bracing around statement
#101 FILE: lib/dpif-plugin.c:52:
            if (dlh == NULL)

ERROR: Use xstrdup() in place of strdup()
#116 FILE: lib/dpif-plugin.c:67:
    p->id = strdup(str);


Please check this out.  If you feel there has been an error, please email 
[email protected]

Thanks,
0-day Robot
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to