Re: Patch: linux-2.5.30/fs/partitions/ldm.c BUG_ON(cond1 || cond2) separation

From: Richard Russon (ldm@flatcap.org)
Date: Mon Aug 05 2002 - 18:19:33 EST


Hi Adam,

> linux-2.5.30/fs/partitions/ldm.c had 23 statements
> of the form BUG_ON(condition1 || condition2). This patch changes
> them to:
>
> BUG_ON(condition1);
> BUG_ON(condition2);

Hmm... The only reason I put the BUG_ONs in, is paranoia. I cannot
imagine how they could be tripped, since all the pointers will have
already been checked. If you wish to standardise the way BUG_ON is
used, then OK, change them.

> Could you please let me know if you want to handle submitting it

Please can you submit the patch to Linus.

Cheers,
  FlatCap (Rich)
  ldm@flatcap.org

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



This archive was generated by hypermail 2b29 : Wed Aug 07 2002 - 22:00:29 EST