Re: [patch] "ntfs_s16 undeclared", missing includefiles

Dale E. Martin (dmartin@clifton-labs.com)
03 Jan 1999 15:13:05 -0500


Arjan van de Ven <arjan@fenrus.demon.nl> writes:

> Hi,
>
> The following patch fixes the new ntfs-driver, it missed two include-files
> so that line 422 of inode.c lead to an unknown ntfs_s16 type. The patch
> fixes this by including the appropriate header-files.
>
> Greetings,
> Arjan van de Ven
>
>
> --- linux/fs/ntfs/inode.c.org Sun Jan 3 11:10:54 1999
> +++ linux/fs/ntfs/inode.c Sun Jan 3 11:11:04 1999
> @@ -18,6 +18,8 @@
> #include "dir.h"
> #include "support.h"
> #include "util.h"
> +#include "ntfstypes.h"
> +#include "ntfsendian.h"
>
> typedef struct {
> int recno;

Hi. This didn't work for me. If you look, those files _are_ included in
inode.c before this patch. (Actually, only one is, but it includes the
other.)

The patch that worked for me was:

*** ntfs_fs_i.h~ Fri Dec 19 18:24:22 1997
--- ntfs_fs_i.h Sun Jan 3 15:09:03 1999
***************
*** 12,17 ****
--- 12,21 ----
typedef unsigned short ntfs_u16;
typedef unsigned int ntfs_u32;
typedef unsigned long long ntfs_u64;
+ typedef s8 ntfs_s8;
+ typedef s16 ntfs_s16;
+ typedef s32 ntfs_s32;
+ typedef s64 ntfs_s64;
#endif

#ifndef NTMODE_T

Enjoy!

Dale

-- 
+------------------------- pgp key available --------------------------+
| Dale E. Martin |  Clifton Labs, Inc.  |  Senior Computer Engineer    |
| dmartin@clifton-labs.com    |    http://www.clifton-labs.com         |
+----------------------------------------------------------------------+

- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.rutgers.edu Please read the FAQ at http://www.tux.org/lkml/