On Thu, Feb 8, 2018 at 3:18 PM, Ben Pfaff <[email protected]> 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 <[email protected]> > ---
Looks good to me. Acked-by: William Tu <[email protected]> > 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 > [email protected] > https://mail.openvswitch.org/mailman/listinfo/ovs-dev _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
