Re: mtab: libdiskfs / ext2fs support changes

[Niels Möller]

David Walter <dwalter@syr.edu> writes:

> +++ ext2fs.c  10 Sep 2002 02:24:24 -0000
> ...
> +extern char*diskfs_mount_point;

It's poor style to declare global variables like this, in a random C
file that happens to use the variable...

> +  /* Prototype for internal routine diskfs_notify_mtab */
> +  void diskfs_notify_mtab(char*);

... and even worse to declare a function that way.

Declarations for global functions and variables should always be put
in some header file which is included both by the file defining the
variable/function in question, and by all files using it. That way,
you get some help from the C compiler to catch type errors.

Hmm, looking at your patch, it seems you already have a declaration of
the diskfs_notify_mtab function in diskfs.h, so you can just delete
the above declaration from extfs.c. As for diskfs_mount_point, if
libdiskfs users like ext2 are supposed to use that variable, the
declaration should move from libdiskfs/priv.h to libdiskfs/diskfs.h.

Best regards,
/Niels