Re: [tip:core/urgent] WARN_ON_SMP(): Allow use in if() statementson UP

From: Steven Rostedt
Date: Fri Mar 25 2011 - 15:53:17 EST


On Fri, 2011-03-25 at 21:47 +0200, Alexey Dobriyan wrote:

> I always thought if (WARN_ON(x)) is confusing and should be banned.

I think that's the preferred method (I use it all the time).

if (WARN_ON(blah))
goto fail;

Better than:

if (x) {
WARN_ON(1);
goto fail;
}

-- Steve


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/