[ovs-dev] [PATCH 3/3] vconn: Remove comma from SSL long options #define

Ben Pfaff blp at nicira.com
Tue Jan 5 07:06:20 UTC 2010


On Mon, Jan 04, 2010 at 09:41:47PM -0800, Justin Pettit wrote:
> The definition of VCONN_SSL_LONG_OPTIONS included a comma at the end of
> its option list.  All other defintions of long options did not include
> this comma, so this commit makes makes it look like the others.

This difference is by design, because VCONN_SSL_LONG_OPTIONS expands
to nothing when OpenSSL isn't available, and if it was followed by a
comma then this would result in two commas in a row, which is a syntax
error.

A comment might be a good idea.




More information about the dev mailing list