coccinelle: remove parentheses that become unnecessary
Transformations that hide multiplications can end up with an pair of parentheses that is no longer needed. E.g. with a rule like this: @@ expression E; @@ - E * 2 + double(E) ... we might get a patch like this: - x = (a + b) * 2; + x = double((a + b)); Add a pair of parentheses to the preimage side of such rules. Coccinelle will generate patches that remove them if they are present, and it will still match expressions that lack them. Signed-off-by: Rene Scharfe <l.s.r@web.de> Signed-off-by: Junio C Hamano <gitster@pobox.com>maint
							parent
							
								
									4010f1d1b7
								
							
						
					
					
						commit
						9ca356fa8b
					
				|  | @ -4,7 +4,7 @@ T *dst; | ||||||
| T *src; | T *src; | ||||||
| expression n; | expression n; | ||||||
| @@ | @@ | ||||||
| - memcpy(dst, src, n * sizeof(*dst)); | - memcpy(dst, src, (n) * sizeof(*dst)); | ||||||
| + COPY_ARRAY(dst, src, n); | + COPY_ARRAY(dst, src, n); | ||||||
|  |  | ||||||
| @@ | @@ | ||||||
|  | @ -13,7 +13,7 @@ T *dst; | ||||||
| T *src; | T *src; | ||||||
| expression n; | expression n; | ||||||
| @@ | @@ | ||||||
| - memcpy(dst, src, n * sizeof(*src)); | - memcpy(dst, src, (n) * sizeof(*src)); | ||||||
| + COPY_ARRAY(dst, src, n); | + COPY_ARRAY(dst, src, n); | ||||||
|  |  | ||||||
| @@ | @@ | ||||||
|  | @ -22,7 +22,7 @@ T *dst; | ||||||
| T *src; | T *src; | ||||||
| expression n; | expression n; | ||||||
| @@ | @@ | ||||||
| - memcpy(dst, src, n * sizeof(T)); | - memcpy(dst, src, (n) * sizeof(T)); | ||||||
| + COPY_ARRAY(dst, src, n); | + COPY_ARRAY(dst, src, n); | ||||||
|  |  | ||||||
| @@ | @@ | ||||||
|  | @ -47,7 +47,7 @@ type T; | ||||||
| T *ptr; | T *ptr; | ||||||
| expression n; | expression n; | ||||||
| @@ | @@ | ||||||
| - ptr = xmalloc(n * sizeof(*ptr)); | - ptr = xmalloc((n) * sizeof(*ptr)); | ||||||
| + ALLOC_ARRAY(ptr, n); | + ALLOC_ARRAY(ptr, n); | ||||||
|  |  | ||||||
| @@ | @@ | ||||||
|  | @ -55,5 +55,5 @@ type T; | ||||||
| T *ptr; | T *ptr; | ||||||
| expression n; | expression n; | ||||||
| @@ | @@ | ||||||
| - ptr = xmalloc(n * sizeof(T)); | - ptr = xmalloc((n) * sizeof(T)); | ||||||
| + ALLOC_ARRAY(ptr, n); | + ALLOC_ARRAY(ptr, n); | ||||||
|  |  | ||||||
		Loading…
	
		Reference in New Issue
	
	 René Scharfe
						René Scharfe