[ovs-dev] [ovsdb-idlc 1/4] ovsdb-idlc: Use column name for "*_set()" smap arguments.

Ben Pfaff blp at nicira.com
Wed Mar 4 21:26:27 UTC 2015


On Wed, Mar 04, 2015 at 11:25:50AM -0800, Justin Pettit wrote:
> Signed-off-by: Justin Pettit <jpettit at nicira.com>

I agree that this is more user-friendly but it runs the risk that
someone will name an smap column "row" or "datum" or "auto" or
"switch" or "goto" or "double" or ...

Maybe that isn't a realistic problem.  At least we'd get a good error
message from the compiler.  That's good enough for me I guess.

Acked-by: Ben Pfaff <blp at nicira.com>



More information about the dev mailing list