Btrfs devel - Dec 2008 - [PATCH] __must_check btrfs_alloc_path()

IMHO it''s a good thing to add __must_check in new code where it has
sense.


Index: btrfs-unstable-standalone/ctree.c
==================================================================---
btrfs-unstable-standalone.orig/ctree.c	2008-12-29 21:57:15.366588465 +0100
+++ btrfs-unstable-standalone/ctree.c	2008-12-29 21:58:06.878588402 +0100
@@ -43,7 +43,7 @@
 	memset(p, 0, sizeof(*p));
 }
 
-struct btrfs_path *btrfs_alloc_path(void)
+struct btrfs_path __must_check *btrfs_alloc_path(void)
 {
 	struct btrfs_path *path;
 	path = kmem_cache_alloc(btrfs_path_cachep, GFP_NOFS);
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

On Mon, 2008-12-29 at 22:02 +0100, Diego Calleja wrote:
> IMHO it''s a good thing to add __must_check in new code where it
has sense.
> 
> 
I don''t think we''ll pass this one yet ;)  But you''re
right that it is a
good idea to start adding it to other functions.

-chris


--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs"
in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html