On Thu, Feb 8, 2018 at 3:18 PM, Ben Pfaff <b...@ovn.org> wrote: > This removes the requirement of exactly two spaces before the error > description (now one or more is fine). It also makes an error message > clearer. > > Signed-off-by: Ben Pfaff <b...@ovn.org> > ---
Looks good to me. Acked-by: William Tu <u9012...@gmail.com> > build-aux/extract-ofp-errors | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/build-aux/extract-ofp-errors b/build-aux/extract-ofp-errors > index 71ae0bd5f0ff..6c14f68b883a 100755 > --- a/build-aux/extract-ofp-errors > +++ b/build-aux/extract-ofp-errors > @@ -247,7 +247,7 @@ def extract_ofp_errors(fn): > expected_errors[m.group(1)] = (fileName, lineNumber) > continue > > - m = re.match('((?:.(?!\. ))+.)\. (.*)$', comment) > + m = re.match('((?:.(?!\. ))+.)\.\s+(.*)$', comment) > if not m: > fatal("unexpected syntax between errors") > > @@ -257,7 +257,7 @@ def extract_ofp_errors(fn): > m = re.match('\s+(?:OFPERR_([A-Z0-9_]+))(\s*=\s*OFPERR_OFS)?,', > line) > if not m: > - fatal("syntax error expecting enum value") > + fatal("syntax error expecting OFPERR_ enum value") > > enum = m.group(1) > if enum in names: > -- > 2.15.1 > > _______________________________________________ > dev mailing list > d...@openvswitch.org > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev