[PATCH v2 4/9] bfs: coding style cleanup in fs/bfs/dir.c

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Dmitri Vorobiev
Date: Friday, January 25, 2008 - 2:20 pm

Clean up errors found by checkpatch.pl.

Before the patch:

$ ./scripts/checkpatch.pl --file  fs/bfs/dir.c | grep total
total: 7 errors, 1 warnings, 370 lines checked

After the patch:

$ ./scripts/checkpatch.pl --file  fs/bfs/dir.c | grep total
total: 0 errors, 1 warnings, 370 lines checked

No functional changes introduced.

This patch was compile-tested by building the BFS driver both
as a module and as a part of the kernel proper.

Signed-off-by: Dmitri Vorobiev <dmitri.vorobiev@gmail.com>
Acked-by: Tigran Aivazian <tigran@aivazian.fsnet.co.uk>
---
 fs/bfs/dir.c |   14 +++++++-------
 1 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/fs/bfs/dir.c b/fs/bfs/dir.c
index 1fd056d..5462a5b 100644
--- a/fs/bfs/dir.c
+++ b/fs/bfs/dir.c
@@ -72,7 +72,7 @@ static int bfs_readdir(struct file *f, void *dirent, filldir_t filldir)
 	}
 
 	unlock_kernel();
-	return 0;	
+	return 0;
 }
 
 const struct file_operations bfs_dir_operations = {
@@ -117,7 +117,7 @@ static int bfs_create(struct inode *dir, struct dentry *dentry, int mode,
 	BFS_I(inode)->i_sblock = 0;
 	BFS_I(inode)->i_eblock = 0;
 	insert_inode_hash(inode);
-        mark_inode_dirty(inode);
+	mark_inode_dirty(inode);
 	dump_imap("create", s);
 
 	err = bfs_add_entry(dir, dentry->d_name.name, dentry->d_name.len,
@@ -228,8 +228,8 @@ static int bfs_rename(struct inode *old_dir, struct dentry *old_dentry,
 		return -EINVAL;
 
 	lock_kernel();
-	old_bh = bfs_find_entry(old_dir, 
-				old_dentry->d_name.name, 
+	old_bh = bfs_find_entry(old_dir,
+				old_dentry->d_name.name,
 				old_dentry->d_name.len, &old_de);
 
 	if (!old_bh || (le16_to_cpu(old_de->ino) != old_inode->i_ino))
@@ -237,8 +237,8 @@ static int bfs_rename(struct inode *old_dir, struct dentry *old_dentry,
 
 	error = -EPERM;
 	new_inode = new_dentry->d_inode;
-	new_bh = bfs_find_entry(new_dir, 
-				new_dentry->d_name.name, 
+	new_bh = bfs_find_entry(new_dir,
+				new_dentry->d_name.name,
 				new_dentry->d_name.len, &new_de);
 
 	if (new_bh && !new_inode) {
@@ -246,7 +246,7 @@ static int bfs_rename(struct inode *old_dir, struct dentry *old_dentry,
 		new_bh = NULL;
 	}
 	if (!new_bh) {
-		error = bfs_add_entry(new_dir, 
+		error = bfs_add_entry(new_dir,
 					new_dentry->d_name.name,
 					new_dentry->d_name.len,
 					old_inode->i_ino);
-- 
1.5.3

-
To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH v2 0/9] bfs: assorted cleanups, Dmitri Vorobiev, (Fri Jan 25, 2:20 pm)
[PATCH v2 1/9] bfs: remove a useless variable, Dmitri Vorobiev, (Fri Jan 25, 2:20 pm)
[PATCH v2 2/9] bfs: coding style cleanup in fs/bfs/inode.c, Dmitri Vorobiev, (Fri Jan 25, 2:20 pm)
[PATCH v2 3/9] bfs: coding style cleanup in fs/bfs/bfs.h, Dmitri Vorobiev, (Fri Jan 25, 2:20 pm)
[PATCH v2 4/9] bfs: coding style cleanup in fs/bfs/dir.c, Dmitri Vorobiev, (Fri Jan 25, 2:20 pm)
[PATCH v2 6/9] bfs: coding style cleanup in fs/bfs/file.c, Dmitri Vorobiev, (Fri Jan 25, 2:20 pm)
[PATCH v2 8/9] bfs: remove multiple assignments, Dmitri Vorobiev, (Fri Jan 25, 2:20 pm)
[PATCH v2 9/9] bfs: use the proper header file for inclusion, Dmitri Vorobiev, (Fri Jan 25, 2:20 pm)
Re: [PATCH v2 8/9] bfs: remove multiple assignments, Tigran Aivazian, (Sat Jan 26, 11:35 am)
Re: [PATCH v2 8/9] bfs: remove multiple assignments, Dmitri Vorobiev, (Sat Jan 26, 2:32 pm)
Re: [PATCH v2 8/9] bfs: remove multiple assignments, Dmitri Vorobiev, (Sat Jan 26, 4:48 pm)
Re: [PATCH v2 8/9] bfs: remove multiple assignments, Adrian Bunk, (Sun Jan 27, 7:08 am)
Re: [PATCH v2 8/9] bfs: remove multiple assignments, Dmitri Vorobiev, (Sun Jan 27, 7:39 am)
Re: [PATCH v2 8/9] bfs: remove multiple assignments, Joel Schopp, (Mon Jan 28, 12:02 am)
Re: [PATCH v2 8/9] bfs: remove multiple assignments, Dmitri Vorobiev, (Wed Jan 30, 6:36 am)