I see at least 6 completely obvious violations of coding style, plus another one that GCC should report for you as a warning. If you cannot see these problems, after comparing the code you wrote to the existing code and carefully reading CodingStyle, then you should find someone else to proofread your patches before you send them.
On Mon, Jul 30, 2012 at 03:55:34AM +0000, Amit Tewari wrote: > Hi Ben, > I will test the patch on master version. > > I followed the 'CodingStyle' document available with openvswitch-1.6.1 tar > for coding guidelines. > Please let me know what guidelines are not followed in vconn.c so that I can > fix that. > > > Regards > Amit Tewari > NEC-HCL ST > > > -----Original Message----- > From: Ben Pfaff [mailto:b...@nicira.com] > Sent: Friday, July 27, 2012 10:31 PM > To: Amit Tewari > Cc: 'discuss@openvswitch.org'; 'b...@openvswitch.org' > Subject: Re: [PATCH] lib/ofp-errors.h, lib/vconn.c,, lib/ofp-errors.c:: > OFPERR_OFPBRC_BAD_VERSION generated from switch when there is version > mismatch. > > On Fri, Jul 27, 2012 at 12:05:07PM +0000, Amit Tewari wrote: > > I am resending patch for generating OFPERR_OFPBRC_BAD_VERSION error > > in case of version mismatch in openflow packets from controller to > > switch. This patch is for latest release 1.6.1. > > We only accept patches for "master". > > The changes to vconn.c don't follow the coding style at all. > > > > DISCLAIMER: > > ----------------------------------------------------------------------------------------------------------------------- > > The contents of this e-mail and any attachment(s) are confidential and > intended > > for the named recipient(s) only. > > It shall not attach any liability on the originator or NECHCL or its > > affiliates. Any views or opinions presented in > > this email are solely those of the author and may not necessarily reflect the > > opinions of NECHCL or its affiliates. > > Any form of reproduction, dissemination, copying, disclosure, modification, > > distribution and / or publication of > > this message without the prior written consent of the author of this e-mail is > > strictly prohibited. If you have > > received this email in error please delete it and notify the sender > > immediately. . > > ----------------------------------------------------------------------------------------------------------------------- _______________________________________________ discuss mailing list discuss@openvswitch.org http://openvswitch.org/mailman/listinfo/discuss
