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

From: Robert P. J. Day
Date: Fri Feb 02 2007 - 07:14:00 EST


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@xxxxxxxxxxxxxx>
> >
> > ---
> >
> > 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@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/