]> bbs.cooldavid.org Git - net-next-2.6.git/commit - mm/vmalloc.c
mm: use ERR_CAST
authorJulia Lawall <julia@diku.dk>
Tue, 10 Aug 2010 00:18:28 +0000 (17:18 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Tue, 10 Aug 2010 03:44:54 +0000 (20:44 -0700)
commite7d86340793e7162126926ec9d226c68f4e37f94
tree4cc90e5466a69b361f1a6eecc2d9fac77e2e1612
parent90d7404558fbe6f369d5e27b5ea3ef1e57562d3d
mm: 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.

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>
Cc: Nick Piggin <nickpiggin@yahoo.com.au>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
mm/vmalloc.c