Re: [PATCH 62/72] drm/i915: split image_caches array into struct for better readability
Lawrence D'Oliveiro <[email protected]>
| Newsgroups | gmane.comp.lib.cairo |
|---|---|
| Organization | Geek Central |
| Message-ID | <[email protected]> |
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. -- cairo mailing list [email protected] http://lists.cairographics.org/mailman/listinfo/cairo