diff options
author | Alexey Dobriyan <adobriyan@gmail.com> | 2006-09-29 02:01:05 -0700 |
---|---|---|
committer | Linus Torvalds <torvalds@g5.osdl.org> | 2006-09-29 09:18:20 -0700 |
commit | 368bdb3d616fa352971f45b423ae6344715e620b (patch) | |
tree | dcb02bd584d5d23c12393726f91694b1fc03ccc1 | |
parent | a4376e13ce07774be14e83fe501ef5c8500b83a1 (diff) | |
download | lwn-368bdb3d616fa352971f45b423ae6344715e620b.tar.gz lwn-368bdb3d616fa352971f45b423ae6344715e620b.zip |
[PATCH] cramfs: make cramfs_uncompress_exit() return void
It always returns 0, so relying on it is useless. The only caller isn't
checking return value. In general, un-, de-, -free functions should return
void.
Signed-off-by: Alexey Dobriyan <adobriyan@gmail.com>
Signed-off-by: Andrew Morton <akpm@osdl.org>
Signed-off-by: Linus Torvalds <torvalds@osdl.org>
-rw-r--r-- | fs/cramfs/uncompress.c | 3 | ||||
-rw-r--r-- | include/linux/cramfs_fs.h | 2 |
2 files changed, 2 insertions, 3 deletions
diff --git a/fs/cramfs/uncompress.c b/fs/cramfs/uncompress.c index 8def89f2c438..fc3ccb74626f 100644 --- a/fs/cramfs/uncompress.c +++ b/fs/cramfs/uncompress.c @@ -68,11 +68,10 @@ int cramfs_uncompress_init(void) return 0; } -int cramfs_uncompress_exit(void) +void cramfs_uncompress_exit(void) { if (!--initialized) { zlib_inflateEnd(&stream); vfree(stream.workspace); } - return 0; } diff --git a/include/linux/cramfs_fs.h b/include/linux/cramfs_fs.h index a41f38428c37..1dba681e428d 100644 --- a/include/linux/cramfs_fs.h +++ b/include/linux/cramfs_fs.h @@ -87,6 +87,6 @@ struct cramfs_super { /* Uncompression interfaces to the underlying zlib */ int cramfs_uncompress_block(void *dst, int dstlen, void *src, int srclen); int cramfs_uncompress_init(void); -int cramfs_uncompress_exit(void); +void cramfs_uncompress_exit(void); #endif |