fs/ceph: Use ERR_CAST
authorJulia Lawall <julia@diku.dk>
Sat, 22 May 2010 10:01:14 +0000 (12:01 +0200)
committerSage Weil <sage@newdream.net>
Sat, 29 May 2010 16:12:41 +0000 (09:12 -0700)
commit7e34bc524ecae3a04d8cc427ee76ddad826a937b
tree5065eb0e287e450aef2d8e03e3a18d8f895a4c9e
parenta41359fa355e7b450c610ed8e913d5d75c3c9c3b
fs/ceph: Use ERR_CAST

Use ERR_CAST(x) rather than ERR_PTR(PTR_ERR(x)).  The former makes more
clear what is the purpose of the operation, which otherwise looks like a
no-op.

In the case of fs/ceph/inode.c, ERR_CAST is not needed, because the type of
the returned value is the same as the type of the enclosing function.

The semantic patch that makes this change is as follows:
(http://coccinelle.lip6.fr/)

// <smpl>
@@
type T;
T x;
identifier f;
@@

T f (...) { <+...
- ERR_PTR(PTR_ERR(x))
+ x
 ...+> }

@@
expression x;
@@

- ERR_PTR(PTR_ERR(x))
+ ERR_CAST(x)
// </smpl>

Signed-off-by: Julia Lawall <julia@diku.dk>
Signed-off-by: Sage Weil <sage@newdream.net>
fs/ceph/dir.c
fs/ceph/export.c
fs/ceph/file.c
fs/ceph/inode.c
fs/ceph/osdmap.c
fs/ceph/super.c