On Fri, May 14, 2021 at 01:05:31PM +0200, Ilya Maximets wrote:
> On 5/12/21 10:15 PM, Ben Pfaff wrote:
> > Signed-off-by: Ben Pfaff <[email protected]>
> > ---
> > v1->v2: Break bond admissibility step into two steps to make it clearer.
> >   Fix a typo.  Rephrase some text for clarity.  Thanks to Ilya Maximets
> >   for the review.
> > 
> >  lib/ovs-actions.xml | 304 +++++++++++++++++++++++++++++++++++++++++++-
> >  1 file changed, 302 insertions(+), 2 deletions(-)
> 
> Thanks!  This version is much more clear.
> 
> There are few typos and extra words:
> 
>   s/lookupn/lookup/
>   s/Accept the packet only if and only it/Accept the packet only if it/
>   s/gratuituous/gratuitous/
>   s/the output port set is the just the/the output port set is just the/
>   s/woule/would/
> 
> Otherwise,
> Acked-by: Ilya Maximets <[email protected]>

Thanks so much for the review and the fixes.  I guess I need a better
static checker for English.  I applied this.
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to