ok, I added peter to the commit message. I'll go ahead and merge. Ethan
On Thu, Feb 17, 2011 at 1:15 PM, Ben Pfaff <[email protected]> wrote: > On Thu, Feb 17, 2011 at 01:07:49PM -0800, Ethan Jackson wrote: >> > I don't think that the change to main() is required, because I think >> > that the 'data' member inside this shash is always NULL. >> >> I would tend to think we should leave it as free_data in main because >> it's that way everywhere else so it sets a precedent. Seems like >> defensive coding. I'm fine with switching it back though if you think >> it matters. > > I don't care. > > Eventually I'm going to implement a "sset" that's just a set of > strings for situations like this. > _______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
