lkml.org 
[lkml]   [2004]   [Oct]   [30]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [2.6 patch] some NTFS cleanups
Hi Adrian,

From a quick read of your patch:

On Sat, 30 Oct 2004, Adrian Bunk wrote:
> The patch below does the following cleanups for the NTFS code:
> - remove three currently unused global functions

Assuming these are the functions in unistr.c then they need to stay. We
are not using them yet but we will when we start creating/deleting files
and things like that.

> - make several functions and variables static (yes, I've read the

Most of those look good, again except the entirety of unistr.c where we
will be using those functions later on. Admittedly we can make them
static for now and undo each as it gets a user outside of unistr.c.

> comment before ntfs_readpage before making it static - but I couldn't
> see it being actually true)

Yes, it was used until a few patches ago - I rewrote mft record writing
code so I no longer needed to define a separate address space ops for
$MFT/$DATA access. I had forgotten about this so the comment was out of
date and the function definitely can be made static now.

> Is this patch OK or does it conflict with your future plans for the NTFS
> code?

Mostly it is fine; great in fact. I will apply it to the ntfs-2.6-devel
tree on Monday but I will leave the three functions you have taken out in.
If you want send me a new patch with the three functions left in before
Monday and I can apply it as is. (-:

Thanks for your work!

Best regards,

Anton
--
Anton Altaparmakov <aia21 at cam.ac.uk> (replace at with @)
Unix Support, Computing Service, University of Cambridge, CB2 3QH, UK
Linux NTFS maintainer / IRC: #ntfs on irc.freenode.net
WWW: http://linux-ntfs.sf.net/ & http://www-stu.christs.cam.ac.uk/~aia21/
-
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/

\
 
 \ /
  Last update: 2005-03-22 14:07    [W:0.029 / U:2.032 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site