[ovs-dev] [PATCH 10/47] ofp-util: Allow encoding of Open Flow 1.2 Flow Mod messages

Ben Pfaff blp at nicira.com
Tue Aug 7 06:21:30 UTC 2012


On Tue, Aug 07, 2012 at 03:16:30PM +0900, Simon Horman wrote:
> On Mon, Aug 06, 2012 at 11:03:58PM -0700, Ben Pfaff wrote:
> > On Wed, Aug 01, 2012 at 04:01:53PM +0900, Simon Horman wrote:
> > > Signed-off-by: Simon Horman <horms at verge.net.au>
> > 
> > I moved the ? to the second line here to match my preferred style:
> > > +    return htons(protocol & OFPUTIL_P_TID ?
> > > +                 (fm->command & 0xff) | (fm->table_id << 8)
> > > +                 : fm->command);
> > 
> > I pushed patches 4-10 to master.  So now, of 1-10, only 3 is not on
> > master.  I'm done for tonight with applying patches, for what it's
> > worth.
> 
> Thanks. I'll clean up 3 and send a rebased series.

Thanks!



More information about the dev mailing list