On Wed, 23 Jul 2014 10:38:34 +0800 Ruoyu <liangry@xxxxxxxxx> wrote: > The statement > > if ((x == f()) < 0) {} > > is strange. I think it should be > > if ((x = f()) < 0) {} > > For the reasone of coding style, I modify it as > > x = f(); > if (x < 0) {} > > Signed-off-by: Ruoyu <liangry@xxxxxxxxx> > --- > usr/bs_rbd.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) Applied, thanks. -- To unsubscribe from this list: send the line "unsubscribe stgt" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html