On Tue, Jun 15, 2021 at 09:28:25AM -0400, Mark Michelson wrote: > On 6/15/21 9:24 AM, Mark Michelson wrote: > > Signed-off-by: Mark Michelson <[email protected]> > > --- > > controller/ofctrl.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/controller/ofctrl.c b/controller/ofctrl.c > > index 053631590..48d001506 100644 > > --- a/controller/ofctrl.c > > +++ b/controller/ofctrl.c > > @@ -1259,7 +1259,6 @@ ofctrl_flood_remove_flows(struct > > ovn_desired_flow_table *flow_table, > > * Copying the sb_uuids into an array. */ > > struct uuid *sb_uuids; > > sb_uuids = xmalloc(hmap_count(flood_remove_nodes) * sizeof *sb_uuids); > > - struct hmap flood_remove_uuids = HMAP_INITIALIZER(&flood_remove_uuids); > > HMAP_FOR_EACH (ofrn, hmap_node, flood_remove_nodes) { > > sb_uuids[n++] = ofrn->sb_uuid; > > } > > > > This series is probably the most trivial I've ever put up for review. I > almost merged this series directly without first doing a review. However, I > talked myself into going through the review process. I'm curious how people > would feel if devs committed changes like this directly instead of engaging > in the review process.
I normally post them for review anyway. The only commits I routinely push without review are ones that just add people to AUTHORS. _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
