lkml.org 
[lkml]   [2002]   [Jan]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [CFT] [JANITORIAL] Unbork fs.h
Hi Daniel,

On Thu, 3 Jan 2002 13:47:01 +0100, Daniel Phillips <phillips@bonn-fries.net> wrote:

> +static struct file_system_type ext2_fs = {
> + owner: THIS_MODULE,
> + fs_flags: FS_REQUIRES_DEV,
> + name: "ext2",
> + read_super: ext2_read_super,
> + super_size: sizeof(struct ext2_sb_info),
> + inode_size: sizeof(struct ext2_inode_info)
> +};

While we're at it, can we extend this model to also include details about
the other filesystem data structures with (potential) private info, i.e.
struct dentry and struct file? ext2 might not use them, but other
filesystems certainly do.

> -static inline struct inode * new_inode(struct super_block *sb)
> +static inline struct inode *new_inode (struct super_block *sb)

Minor issue of coding style. I'd steer away from such gratuitious changes,
especially since they divert from the commonly accepted practice of having
no spaces between the name of the function and its arguments.

Thanks,
-Ion
[FiST co-maintainer]
-
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 13:15    [W:1.496 / U:2.092 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site