On Feb 23, 2011, at 9:30 AM, Ben Pfaff wrote: > On Tue, Feb 22, 2011 at 10:18 PM, Justin Pettit <jpet...@nicira.com> wrote: >> Here are some additional fixes for issues that Coverity flagged. With >> this series, the remaining issues Coverity marks as "High Impact" that >> appear to me to be real are addressed.* This also addresses most of the >> issues that Coverity marks as "Medium Impact" that looked real to me, >> but there are a few remaining ones that I will take another crack at >> later (unless someone beats me to it). > > Thanks a lot for doing this. > > I have one suggestion for any possible future series: order it in rough > descending order of importance, with serious bugs at the beginning and > e.g. memory leaks in test programs at the end. (I try to do this when > I compose patch series, too.)
Sounds good. I'll do that in the future. Thanks for the reviews. I pushed. --Justin _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev_openvswitch.org
