Am 01.11.2013 13:25, schrieb Nikolay Aleksandrov: > On 11/01/2013 12:35 PM, Jiri Pirko wrote: >> Fri, Nov 01, 2013 at 11:41:00AM CET, vfalico@xxxxxxxxxx wrote: >>> On Fri, Nov 01, 2013 at 01:18:44PM +0300, Dan Carpenter wrote: >>>> There is an extra semi-colon so bond_get_size() doesn't return the >>>> correct value. >>>> >>>> Fixes: ec76aa49855f ('bonding: add Netlink support active_slave option') >>>> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> >>> >>> Hrm, how does it even build. Good catch, thanks. >> >> I wonder about the same thing. >> > + and - can be unary operators so it's a valid expression :-) > Nice catch > NTL my gcc warns (-Wall) warning: statement with no effect this should come up. re, wh > Nik > > -- > To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html > -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html