Re: [Mesa-dev] [PATCH 11/19] anv: Use GNU C empty brace initializer

2017-09-14 Thread Emil Velikov
On 13 September 2017 at 23:31, Ian Romanick wrote: > On 08/29/2017 10:19 AM, Emil Velikov wrote: >> On 29 August 2017 at 18:10, Matt Turner wrote: >>> On Tue, Aug 29, 2017 at 3:35 AM, Emil Velikov >>> wrote: On 29 August 2017 at 11:11, Eric Engestrom wrote: > On Monday, 2017-08-

Re: [Mesa-dev] [PATCH 11/19] anv: Use GNU C empty brace initializer

2017-09-13 Thread Ian Romanick
On 08/29/2017 10:19 AM, Emil Velikov wrote: > On 29 August 2017 at 18:10, Matt Turner wrote: >> On Tue, Aug 29, 2017 at 3:35 AM, Emil Velikov >> wrote: >>> On 29 August 2017 at 11:11, Eric Engestrom >>> wrote: On Monday, 2017-08-28 14:57:13 -0700, Matt Turner wrote: > Avoids Clang's w

Re: [Mesa-dev] [PATCH 11/19] anv: Use GNU C empty brace initializer

2017-08-29 Thread Eric Engestrom
On 29 August 2017 18:11:45 BST, Matt Turner wrote: > On Tue, Aug 29, 2017 at 3:11 AM, Eric Engestrom > wrote: > > On Monday, 2017-08-28 14:57:13 -0700, Matt Turner wrote: > >> Avoids Clang's warning about the current code: > >> > >>warning: suggest braces around initialization of subobject >

Re: [Mesa-dev] [PATCH 11/19] anv: Use GNU C empty brace initializer

2017-08-29 Thread Emil Velikov
On 29 August 2017 at 18:10, Matt Turner wrote: > On Tue, Aug 29, 2017 at 3:35 AM, Emil Velikov > wrote: >> On 29 August 2017 at 11:11, Eric Engestrom wrote: >>> On Monday, 2017-08-28 14:57:13 -0700, Matt Turner wrote: Avoids Clang's warning about the current code: warning: sug

Re: [Mesa-dev] [PATCH 11/19] anv: Use GNU C empty brace initializer

2017-08-29 Thread Matt Turner
On Tue, Aug 29, 2017 at 3:11 AM, Eric Engestrom wrote: > On Monday, 2017-08-28 14:57:13 -0700, Matt Turner wrote: >> Avoids Clang's warning about the current code: >> >>warning: suggest braces around initialization of subobject > > I'm not sure about this patch [1], but everything else in this

Re: [Mesa-dev] [PATCH 11/19] anv: Use GNU C empty brace initializer

2017-08-29 Thread Matt Turner
On Tue, Aug 29, 2017 at 3:35 AM, Emil Velikov wrote: > On 29 August 2017 at 11:11, Eric Engestrom wrote: >> On Monday, 2017-08-28 14:57:13 -0700, Matt Turner wrote: >>> Avoids Clang's warning about the current code: >>> >>>warning: suggest braces around initialization of subobject >> >> I'm n

Re: [Mesa-dev] [PATCH 11/19] anv: Use GNU C empty brace initializer

2017-08-29 Thread Emil Velikov
On 29 August 2017 at 11:11, Eric Engestrom wrote: > On Monday, 2017-08-28 14:57:13 -0700, Matt Turner wrote: >> Avoids Clang's warning about the current code: >> >>warning: suggest braces around initialization of subobject > > I'm not sure about this patch [1], but everything else in this seri

Re: [Mesa-dev] [PATCH 11/19] anv: Use GNU C empty brace initializer

2017-08-29 Thread Eric Engestrom
On Monday, 2017-08-28 14:57:13 -0700, Matt Turner wrote: > Avoids Clang's warning about the current code: > >warning: suggest braces around initialization of subobject I'm not sure about this patch [1], but everything else in this series is: Reviewed-by: Eric Engestrom The "mark `UNUSED` be

[Mesa-dev] [PATCH 11/19] anv: Use GNU C empty brace initializer

2017-08-28 Thread Matt Turner
Avoids Clang's warning about the current code: warning: suggest braces around initialization of subobject --- src/intel/vulkan/anv_formats.c | 2 +- src/intel/vulkan/anv_pipeline.c | 20 ++-- 2 files changed, 11 insertions(+), 11 deletions(-) diff --git a/src/intel/vulkan/an