"dev" <[email protected]> wrote on 08/08/2016 11:14:14 AM:

> From: Ben Pfaff <[email protected]>
> To: [email protected]
> Cc: Ben Pfaff <[email protected]>
> Date: 08/08/2016 11:15 AM
> Subject: [ovs-dev] [PATCH v2 03/21] expr: Fine-tune parser error
> message for common typo.
> Sent by: "dev" <[email protected]>
>
> It's easy to type "=" in place of "==" in an expression but the
expression
> parser's error message was far from clear.  For multibit numeric fields,
> it said:
>     Explicit `!= 0' is required for inequality test of multibit field
>     against 0.
> For string fields, the parser treated such an expression as "<name> != 0"
> and thus it said:
>     String field <name> is not compatible with numeric constant.
>
> This improves the error message in each case to:
>     Syntax error at `=' expecting relational operator.
> which I hope to be clear.
>
> Signed-off-by: Ben Pfaff <[email protected]>
> ---

I'm pretty much always a sucker for better error messages...

Acked-by: Ryan Moats <[email protected]>
_______________________________________________
dev mailing list
[email protected]
http://openvswitch.org/mailman/listinfo/dev

Reply via email to