[cairo] [PATCH 62/72] drm/i915: split image_caches array into struct for better readability

Lawrence D'Oliveiro ldo at geek-central.gen.nz
Tue Dec 29 15:30:50 PST 2015


On Tue, 29 Dec 2015 16:46:49 +0100, Enrico Weigelt, metux IT consult
wrote:

> On 29.12.2015 16:04, Chris Wilson wrote:
> 
> >> -	caches = &device->image_caches[1];
> >> +	caches = &(device->image_caches.a8);  
> > 
> > Please reduce the number of random brackets in the patches.  
> 
> The brackets after the & ?
> I intentionally added them to make it more clear to the human
> reader, what we're going to reference here.

I’m trying to understand how many possible interpretations you think
the construct *without* the redundant parentheses could have.


More information about the cairo mailing list