Re: [RESEND][PATCH -mm 1/2] vfs: fix coding style at dcache.c.

From: Andrew Morton
Date: Mon Jun 23 2008 - 22:07:31 EST


On Tue, 24 Jun 2008 10:45:14 +0900 Kentaro Makita <k-makita@xxxxxxxxxxxxxxxxxx> wrote:

> o fix coding style at dcache.c. No behavior changed.
>
> Signed-off-by: Kentaro Makita <k-makita@xxxxxxxxxxxxxxxxxx>
> ---
> fs/dcache.c | 21 ++++++++++-----------
> 1 files changed, 10 insertions(+), 11 deletions(-)
>
> Index: b/fs/dcache.c
> ===================================================================
> --- a/fs/dcache.c 2008-06-05 11:44:26.000000000 +0900
> +++ b/fs/dcache.c 2008-06-05 11:48:57.000000000 +0900
> @@ -453,15 +453,15 @@ static void __shrink_dcache_sb(struct su
> int cnt = 0;
>
> BUG_ON(!sb);
> - BUG_ON((flags & DCACHE_REFERENCED) && count == NULL);
> + BUG_ON((flags & DCACHE_REFERENCED) && !count);

A change like this isn't really a "fix". Both styles are acceptable
and some people prefer the longer form.

--
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/