Bleep bloop.  Greetings Mary Manohar, 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: Line lacks whitespace around operator
#71 FILE: ovn/utilities/ovn-nbctl.c:644:
  lr-policy-add ROUTER PRIORITY MATCH ACTION [NEXTHOP]\n\

WARNING: Line lacks whitespace around operator
#73 FILE: ovn/utilities/ovn-nbctl.c:646:
  lr-policy-del ROUTER [PRIORITY [MATCH]]\n\

WARNING: Line lacks whitespace around operator
#75 FILE: ovn/utilities/ovn-nbctl.c:648:
  lr-policy-list ROUTER   print policies for ROUTER\n\

ERROR: Inappropriate bracing around statement
#146 FILE: ovn/utilities/ovn-nbctl.c:3465:
    if (next_hop != NULL)

ERROR: Inappropriate bracing around statement
#232 FILE: ovn/utilities/ovn-nbctl.c:3551:
    } else

Lines checked: 346, Warnings: 3, Errors: 2


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