[PATCH 0/9] bfs: assorted cleanups

Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
From: Dmitri Vorobiev
Date: Thursday, January 24, 2008 - 3:31 pm

Hello Adrian,

Last time when I had sent some BFS bugfixes to the maintainer of
this filesystem driver, he did not respond, and Andrew Morton
had to take care of those.

For this reason, and because the patches in this little patch bomb
are trivial, I am sending this to you in the hope that these can
be pushed upstream when the next merge window opens.

In case you'll be wondering why I need BFS: I teach system programming
and operating systems at the premises of the Moscow State University.
The BFS code is used as an example of a simple filesystem driver
implementation in Linux. This hopefully explains why I want to have
clean code in here :)

Some checkpatch.pl stats follow.

Before:

----------------------------------------
|         | errors | warnings | checks |
----------------------------------------
| bfs.h   |   2    |    0     |   0    |
----------------------------------------
| dir.c   |   7    |    1     |   4    |
----------------------------------------
| file.c  |   6    |    0     |   2    |
----------------------------------------
| inode.c |   11   |    0     |   3    |
----------------------------------------

After:

----------------------------------------
|         | errors | warnings | checks |
----------------------------------------
| bfs.h   |   0    |    0     |   0    |
----------------------------------------
| dir.c   |   0    |    0     |   0    |
----------------------------------------
| file.c  |   0    |    0     |   0    |
----------------------------------------
| inode.c |   0    |    0     |   0    |
----------------------------------------

Please consider.

Thanks,

Dmitri Vorobiev
--
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH 0/9] bfs: assorted cleanups, Dmitri Vorobiev, (Thu Jan 24, 3:31 pm)
[PATCH 1/9] bfs: remove a useless variable, Dmitri Vorobiev, (Thu Jan 24, 3:32 pm)
[PATCH 2/9] bfs: coding style cleanup in fs/bfs/inode.c, Dmitri Vorobiev, (Thu Jan 24, 3:32 pm)
[PATCH 3/9] bfs: coding style cleanup in fs/bfs/bfs.h, Dmitri Vorobiev, (Thu Jan 24, 3:32 pm)
[PATCH 4/9] bfs: coding style cleanup in fs/bfs/dir.c, Dmitri Vorobiev, (Thu Jan 24, 3:32 pm)
[PATCH 6/9] bfs: coding style cleanup in fs/bfs/file.c, Dmitri Vorobiev, (Thu Jan 24, 3:32 pm)
[PATCH 8/9] bfs: remove multiple assignments, Dmitri Vorobiev, (Thu Jan 24, 3:32 pm)
[PATCH 9/9] bfs: use the proper header file for inclusion, Dmitri Vorobiev, (Thu Jan 24, 3:32 pm)
Re: [PATCH 0/9] bfs: assorted cleanups, Dmitri Vorobiev, (Fri Jan 25, 3:25 am)