Re: [ovs-dev] [coverity3 08/13] ovs-vsctl: Simplify parse_column_key_value().

2011-02-23 Thread Ethan Jackson
Looks Good. On Wed, Feb 23, 2011 at 1:24 PM, Ben Pfaff wrote: > Coverity pointed out some inconsistencies on tests for whether columnp and > keyp were nonnull.  These tests were, at best, confusing, but in fact every > caller always passed nonnull for both parameters, so this commit drops all > o

[ovs-dev] [coverity3 08/13] ovs-vsctl: Simplify parse_column_key_value().

2011-02-23 Thread Ben Pfaff
Coverity pointed out some inconsistencies on tests for whether columnp and keyp were nonnull. These tests were, at best, confusing, but in fact every caller always passed nonnull for both parameters, so this commit drops all of those tests. Coverity #10715, 10710. --- utilities/ovs-vsctl.c | 6