[ovs-dev] [cleanups 09/12] bridge: Always "up" internal devices.

Justin Pettit jpettit at nicira.com
Fri Nov 16 18:49:39 UTC 2012


On Nov 16, 2012, at 9:54 AM, Ben Pfaff <blp at nicira.com> wrote:

> On Fri, Nov 16, 2012 at 12:03:02AM -0800, Justin Pettit wrote:
>> The kernel datapath automatically "up"s internal devices, but this
>> wasn't happening for the userspace datapath.  This change has the bridge
>> module always "up" them.
>> 
>> Signed-off-by: Justin Pettit <jpettit at nicira.com>
> 
> Seems reasonable, thanks.
> 
> The indentation here, on the second line, looks odd to me:
>    if ((port_cfg->vlan_mode && !strcmp(port_cfg->vlan_mode, "splinter"))
>            || iface_is_internal(iface_cfg, br->cfg)) {
> I would write it as:
>    if ((port_cfg->vlan_mode && !strcmp(port_cfg->vlan_mode, "splinter"))
>        || iface_is_internal(iface_cfg, br->cfg)) {

I believe that would line it up with the code block within the statement, which looks odd to me.  In our CodingStyle, it seems like there's an example using that similar style where there's two idents on a long if statement.  What do you think?

--Justin





More information about the dev mailing list