Re: [PATCH] NTFS: Rename apparently incorrect macro name "NTFS_DEBUG".

["Robert P. J. Day" <rpjday@mindspring.com>]

On Fri, 2 Feb 2007, Anton Altaparmakov wrote:

> Hi,
>
> On Fri, 2007-02-02 at 05:46 -0500, Robert P. J. Day wrote:
> >   Replace the apparently incorrect macro name "NTFS_DEBUG" with
> > "CONFIG_NTFS_DEBUG".
> >
> > Signed-off-by: Robert P. J. Day <rpjday@mindspring.com>
> >
> > ---
> >
> > diff --git a/fs/ntfs/attrib.c b/fs/ntfs/attrib.c
> > index c577d8e..2154d03 100644
> > --- a/fs/ntfs/attrib.c
> > +++ b/fs/ntfs/attrib.c
> > @@ -1921,7 +1921,7 @@ s64 ntfs_attr_extend_allocation(ntfs_inode *ni, s64 new_alloc_size,
> >  	u32 attr_len = 0; /* Silence stupid gcc warning. */
> >  	bool mp_rebuilt;
> >
> > -#ifdef NTFS_DEBUG
> > +#ifdef CONFIG_NTFS_DEBUG
>
> The fix is also wrong.  It should be:
>
> #ifdef DEBUG
>
> See fs/ntfs/Makefile for details...

ah, yes, i see what you mean.  revised patch coming shortly, i'll send
that one to just the relevant parties.

rday

-- 
========================================================================
Robert P. J. Day
Linux Consulting, Training and Annoying Kernel Pedantry
Waterloo, Ontario, CANADA

http://www.fsdev.dreamhosters.com/wiki/index.php?title=Main_Page
========================================================================
-
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/