Messages in this thread | | | Date | Wed, 28 Nov 2007 17:01:23 -0800 | From | Randy Dunlap <> | Subject | Re: [PATCH 4/3] net/bonding: Adhere to coding style: break line after the if condition |
| |
=?utf-8?q?Ferenc_W=C3=A1gner?= wrote: > Signed-off-by: Ferenc Wágner <wferi@niif.hu>
Acked-by: Randy Dunlap <randy.dunlap@oracle.com>
Thanks.
> --- > Randy Dunlap <randy.dunlap@oracle.com> writes: > > drivers/net/bonding/bond_sysfs.c | 9 ++++++--- > 1 files changed, 6 insertions(+), 3 deletions(-) > > diff --git a/drivers/net/bonding/bond_sysfs.c b/drivers/net/bonding/bond_sysfs.c > index 5c31f5c..9de2c52 100644 > --- a/drivers/net/bonding/bond_sysfs.c > +++ b/drivers/net/bonding/bond_sysfs.c > @@ -91,7 +91,8 @@ static ssize_t bonding_show_bonds(struct class *cls, char *buf) > } > res += sprintf(buf + res, "%s ", bond->dev->name); > } > - if (res) buf[res-1] = '\n'; /* eat the leftover space */ > + if (res) > + buf[res-1] = '\n'; /* eat the leftover space */ > up_read(&(bonding_rwsem)); > return res; > } > @@ -239,7 +240,8 @@ static ssize_t bonding_show_slaves(struct device *d, > res += sprintf(buf + res, "%s ", slave->dev->name); > } > read_unlock(&bond->lock); > - if (res) buf[res-1] = '\n'; /* eat the leftover space */ > + if (res) > + buf[res-1] = '\n'; /* eat the leftover space */ > return res; > } > > @@ -705,7 +707,8 @@ static ssize_t bonding_show_arp_targets(struct device *d, > res += sprintf(buf + res, "%u.%u.%u.%u ", > NIPQUAD(bond->params.arp_targets[i])); > } > - if (res) buf[res-1] = '\n'; /* eat the leftover space */ > + if (res) > + buf[res-1] = '\n'; /* eat the leftover space */ > return res; > } >
-- ~Randy - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |