[ovs-dev] [PATCH v2] Add build checks for portable OpenFlow structure padding and alignment.
Ben Pfaff
blp at nicira.com
Mon Jan 25 18:47:56 UTC 2010
On Fri, Jan 22, 2010 at 01:50:44PM -0800, Justin Pettit wrote:
> This is very impressive...so impressive that I'm going to take on
> faith that all the token parsing is correct. :-)
Oh, it's not. It's just good enough for the headers that it needs to
parse. Try putting, say, a quoted string in your header and watch it
barf.
> I think the convention in Python is to wrap everything in functions
> and then add an "if __name__ == '__main__'" to the bottom of the
> program that calls into those functions. Very minor, but I think it
> makes things a bit clearer.
OK.
This makes an ugly program superficially better, I guess.
> Thanks for pulling this together so quickly. It will be a great
> sanity-check before pushing the OpenFlow 1.0 changes.
Hey, it even *looks* like it was pulled together quickly. (But it
works.)
I'll push this soon.
More information about the dev
mailing list