Add a ternary operator version of the open-coded PTR_RET(). Signed-off-by: Lars-Peter Clausen <lars@metafoo.de> Signed-off-by: Julia Lawall <Julia.Lawall@lip6.fr> Signed-off-by: Michal Marek <mmarek@suse.cz>
		
			
				
	
	
		
			96 lines
		
	
	
	
		
			1.7 KiB
			
		
	
	
	
		
			Text
		
	
	
	
	
	
			
		
		
	
	
			96 lines
		
	
	
	
		
			1.7 KiB
			
		
	
	
	
		
			Text
		
	
	
	
	
	
///
 | 
						|
/// Use PTR_RET rather than if(IS_ERR(...)) + PTR_ERR
 | 
						|
///
 | 
						|
// Confidence: High
 | 
						|
// Copyright: (C) 2012 Julia Lawall, INRIA/LIP6.  GPLv2.
 | 
						|
// Copyright: (C) 2012 Gilles Muller, INRIA/LiP6.  GPLv2.
 | 
						|
// URL: http://coccinelle.lip6.fr/
 | 
						|
// Options: -no_includes -include_headers
 | 
						|
//
 | 
						|
// Keywords: ERR_PTR, PTR_ERR, PTR_RET
 | 
						|
// Version min: 2.6.39
 | 
						|
//
 | 
						|
 | 
						|
virtual context
 | 
						|
virtual patch
 | 
						|
virtual org
 | 
						|
virtual report
 | 
						|
 | 
						|
@depends on patch@
 | 
						|
expression ptr;
 | 
						|
@@
 | 
						|
 | 
						|
- if (IS_ERR(ptr)) return PTR_ERR(ptr); else return 0;
 | 
						|
+ return PTR_RET(ptr);
 | 
						|
 | 
						|
@depends on patch@
 | 
						|
expression ptr;
 | 
						|
@@
 | 
						|
 | 
						|
- if (IS_ERR(ptr)) return PTR_ERR(ptr); return 0;
 | 
						|
+ return PTR_RET(ptr);
 | 
						|
 | 
						|
@depends on patch@
 | 
						|
expression ptr;
 | 
						|
@@
 | 
						|
 | 
						|
- (IS_ERR(ptr) ? PTR_ERR(ptr) : 0)
 | 
						|
+ PTR_RET(ptr)
 | 
						|
 | 
						|
@r1 depends on !patch@
 | 
						|
expression ptr;
 | 
						|
position p1;
 | 
						|
@@
 | 
						|
 | 
						|
* if@p1 (IS_ERR(ptr)) return PTR_ERR(ptr); else return 0;
 | 
						|
 | 
						|
@r2 depends on !patch@
 | 
						|
expression ptr;
 | 
						|
position p2;
 | 
						|
@@
 | 
						|
 | 
						|
* if@p2 (IS_ERR(ptr)) return PTR_ERR(ptr); return 0;
 | 
						|
 | 
						|
@r3 depends on !patch@
 | 
						|
expression ptr;
 | 
						|
position p3;
 | 
						|
@@
 | 
						|
 | 
						|
* IS_ERR@p3(ptr) ? PTR_ERR(ptr) : 0
 | 
						|
 | 
						|
@script:python depends on org@
 | 
						|
p << r1.p1;
 | 
						|
@@
 | 
						|
 | 
						|
coccilib.org.print_todo(p[0], "WARNING: PTR_RET can be used")
 | 
						|
 | 
						|
 | 
						|
@script:python depends on org@
 | 
						|
p << r2.p2;
 | 
						|
@@
 | 
						|
 | 
						|
coccilib.org.print_todo(p[0], "WARNING: PTR_RET can be used")
 | 
						|
 | 
						|
@script:python depends on org@
 | 
						|
p << r3.p3;
 | 
						|
@@
 | 
						|
 | 
						|
coccilib.org.print_todo(p[0], "WARNING: PTR_RET can be used")
 | 
						|
 | 
						|
@script:python depends on report@
 | 
						|
p << r1.p1;
 | 
						|
@@
 | 
						|
 | 
						|
coccilib.report.print_report(p[0], "WARNING: PTR_RET can be used")
 | 
						|
 | 
						|
@script:python depends on report@
 | 
						|
p << r2.p2;
 | 
						|
@@
 | 
						|
 | 
						|
coccilib.report.print_report(p[0], "WARNING: PTR_RET can be used")
 | 
						|
 | 
						|
@script:python depends on report@
 | 
						|
p << r3.p3;
 | 
						|
@@
 | 
						|
 | 
						|
coccilib.report.print_report(p[0], "WARNING: PTR_RET can be used")
 |