Hi,

On Mon, 2007-08-20 at 09:18 -0400, Chris Snook wrote:

> > How's about we just remove that printk?  Do
> > 
> > #define J_ASSERT(e) BUG_ON(e)?
> > 
> > The rest of the kernel seems to be able to cope with that...
> 
> Perfectly fine with me.  I proposed the conditional on the grounds that 
> someone 
> probably had a purpose for the original J_ASSERT macro at some point in 
> history, 
> but if that purpose is long since obsolete, let's just get rid of it.  I'll 
> repost.

It did.  The original J_ASSERT predates BUG() entirely, and was added so
that we got the file/line-no information.  But with the current BUG()
macro, I can't see any reason for J_ASSERT still to try to gather that
information itself.

--Stephen


-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to