[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Fri, Jan 22, 2016 at 07:21:31PM +, Emil Velikov wrote: > On 22 January 2016 at 18:49, Ville Syrjälä > wrote: > > On Fri, Jan 22, 2016 at 05:59:30PM +, Emil Velikov wrote: > > >> Also let's not forget about > >> > >> a.c:17:20: warning: ISO C forbids empty initializer braces

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Fri, Jan 22, 2016 at 05:59:30PM +, Emil Velikov wrote: > On 22 January 2016 at 17:50, Emil Velikov wrote: > > On 22 January 2016 at 17:47, Ville Syrjälä > > wrote: > >> On Fri, Jan 22, 2016 at 05:40:54PM +, Emil Velikov wrote: > >>> On 22 January 2016 at 17:29, Ilia Mirkin wrote: >

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Fri, Jan 22, 2016 at 05:40:54PM +, Emil Velikov wrote: > On 22 January 2016 at 17:29, Ilia Mirkin wrote: > > On Fri, Jan 22, 2016 at 12:18 PM, Marek Olšák wrote: > >> On Fri, Jan 22, 2016 at 6:13 PM, Emil Velikov >> gmail.com> wrote: > >>> On 21 January 2016 at 16:58, Marek Olšák

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 22 January 2016 at 18:49, Ville Syrjälä wrote: > On Fri, Jan 22, 2016 at 05:59:30PM +, Emil Velikov wrote: >> Also let's not forget about >> >> a.c:17:20: warning: ISO C forbids empty initializer braces [-Wpedantic] >> struct foo f = {}; >> ^ > > I long ago

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Fri, Jan 22, 2016 at 6:13 PM, Emil Velikov wrote: > On 21 January 2016 at 16:58, Marek Olšák wrote: >> On Thu, Jan 21, 2016 at 2:09 PM, Emil Velikov >> wrote: >>> On 21 January 2016 at 12:08, Marek Olšák wrote: On Thu, Jan 21, 2016 at 11:51 AM, Emil Velikov >>> gmail.com> wrote:

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 22 January 2016 at 17:50, Emil Velikov wrote: > On 22 January 2016 at 17:47, Ville Syrjälä > wrote: >> On Fri, Jan 22, 2016 at 05:40:54PM +, Emil Velikov wrote: >>> On 22 January 2016 at 17:29, Ilia Mirkin wrote: >>> > On Fri, Jan 22, 2016 at 12:18 PM, Marek Olšák >>> > wrote: >>>

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 22 January 2016 at 17:47, Ville Syrjälä wrote: > On Fri, Jan 22, 2016 at 05:40:54PM +, Emil Velikov wrote: >> On 22 January 2016 at 17:29, Ilia Mirkin wrote: >> > On Fri, Jan 22, 2016 at 12:18 PM, Marek Olšák >> > wrote: >> >> On Fri, Jan 22, 2016 at 6:13 PM, Emil Velikov > >>

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 22 January 2016 at 17:29, Ilia Mirkin wrote: > On Fri, Jan 22, 2016 at 12:18 PM, Marek Olšák wrote: >> On Fri, Jan 22, 2016 at 6:13 PM, Emil Velikov >> wrote: >>> On 21 January 2016 at 16:58, Marek Olšák wrote: On Thu, Jan 21, 2016 at 2:09 PM, Emil Velikov >>> gmail.com> wrote:

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 21 January 2016 at 16:58, Marek Olšák wrote: > On Thu, Jan 21, 2016 at 2:09 PM, Emil Velikov > wrote: >> On 21 January 2016 at 12:08, Marek Olšák wrote: >>> On Thu, Jan 21, 2016 at 11:51 AM, Emil Velikov >> gmail.com> wrote: On 18 January 2016 at 22:53, Marek Olšák wrote: >

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Fri, 2016-01-22 at 12:48 -0500, Ilia Mirkin wrote: > On Fri, Jan 22, 2016 at 12:47 PM, Ville Syrjälä > wrote: > > On Fri, Jan 22, 2016 at 05:40:54PM +, Emil Velikov wrote: > > > On 22 January 2016 at 17:29, Ilia Mirkin > > > wrote: > > > > On Fri, Jan 22, 2016 at 12:18 PM, Marek Olšák

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Fri, Jan 22, 2016 at 12:47 PM, Ville Syrjälä wrote: > On Fri, Jan 22, 2016 at 05:40:54PM +, Emil Velikov wrote: >> On 22 January 2016 at 17:29, Ilia Mirkin wrote: >> > On Fri, Jan 22, 2016 at 12:18 PM, Marek Olšák >> > wrote: >> >> On Fri, Jan 22, 2016 at 6:13 PM, Emil Velikov > >>

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Fri, Jan 22, 2016 at 12:18 PM, Marek Olšák wrote: > On Fri, Jan 22, 2016 at 6:13 PM, Emil Velikov > wrote: >> On 21 January 2016 at 16:58, Marek Olšák wrote: >>> On Thu, Jan 21, 2016 at 2:09 PM, Emil Velikov >>> wrote: On 21 January 2016 at 12:08, Marek Olšák wrote: > On

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Thu, Jan 21, 2016 at 2:09 PM, Emil Velikov wrote: > On 21 January 2016 at 12:08, Marek Olšák wrote: >> On Thu, Jan 21, 2016 at 11:51 AM, Emil Velikov >> wrote: >>> On 18 January 2016 at 22:53, Marek Olšák wrote: Try explaining that to people who have a compulsion to fix them or

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 21 January 2016 at 12:08, Marek Olšák wrote: > On Thu, Jan 21, 2016 at 11:51 AM, Emil Velikov > wrote: >> On 18 January 2016 at 22:53, Marek Olšák wrote: >>> Try explaining that to people who have a compulsion to fix them or >>> argue about them. :) Ignore? REALLY? IGNORE??? >>> >> Now

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Thu, Jan 21, 2016 at 11:51 AM, Emil Velikov wrote: > On 18 January 2016 at 22:53, Marek Olšák wrote: >> Try explaining that to people who have a compulsion to fix them or >> argue about them. :) Ignore? REALLY? IGNORE??? >> > Now that we have a few people off your back can you please point

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 18 January 2016 at 22:53, Marek Olšák wrote: > Try explaining that to people who have a compulsion to fix them or > argue about them. :) Ignore? REALLY? IGNORE??? > Now that we have a few people off your back can you please point out where this triggers warnings ? I've tried multiple times

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 19.01.2016 01:05, Emil Velikov wrote: > > A few of those are already implicit with either Wall or Wextra. Both > of which, imho, are a must have for any serious project. I think -Wextra is generally too noisy for that, but I guess we're now deeply in arguing about taste territory. > But

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Mon, Jan 18, 2016 at 5:05 PM, Emil Velikov wrote: > On 18 January 2016 at 17:43, Marek Olšák wrote: >> On Mon, Jan 18, 2016 at 3:45 PM, Emil Velikov >> wrote: >>> On 15 January 2016 at 17:24, Marek Olšák wrote: On Fri, Jan 15, 2016 at 12:12 PM, Emil Velikov >>> gmail.com> wrote:

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 18 January 2016 at 17:43, Marek Olšák wrote: > On Mon, Jan 18, 2016 at 3:45 PM, Emil Velikov > wrote: >> On 15 January 2016 at 17:24, Marek Olšák wrote: >>> On Fri, Jan 15, 2016 at 12:12 PM, Emil Velikov >> gmail.com> wrote: On 12 January 2016 at 23:14, Marek Olšák wrote: >

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Mon, 18 Jan 2016, Marek Olšák wrote: > On Mon, Jan 18, 2016 at 3:45 PM, Emil Velikov > wrote: >> On 15 January 2016 at 17:24, Marek Olšák wrote: >>> On Fri, Jan 15, 2016 at 12:12 PM, Emil Velikov >> gmail.com> wrote: On 12 January 2016 at 23:14, Marek Olšák wrote: > From:

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 15 January 2016 at 17:24, Marek Olšák wrote: > On Fri, Jan 15, 2016 at 12:12 PM, Emil Velikov > wrote: >> On 12 January 2016 at 23:14, Marek Olšák wrote: >>> From: Marek Olšák >>> >>> It warns for all "{}" initializers. Well, I want us to use {}. >>> --- >>> configure.ac | 3

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Mon, Jan 18, 2016 at 3:45 PM, Emil Velikov wrote: > On 15 January 2016 at 17:24, Marek Olšák wrote: >> On Fri, Jan 15, 2016 at 12:12 PM, Emil Velikov >> wrote: >>> On 12 January 2016 at 23:14, Marek Olšák wrote: From: Marek Olšák It warns for all "{}" initializers.

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

Hi On Fri, Jan 15, 2016 at 4:24 PM, Marek Olšák wrote: > On Fri, Jan 15, 2016 at 12:12 PM, Emil Velikov > wrote: >> On 12 January 2016 at 23:14, Marek Olšák wrote: >>> From: Marek Olšák >>> >>> It warns for all "{}" initializers. Well, I want us to use {}. >>> --- >>> configure.ac

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

Reviewed-by: Ilia Mirkin On Tue, Jan 12, 2016 at 4:14 PM, Marek Olšák wrote: > From: Marek Olšák > > It warns for all "{}" initializers. Well, I want us to use {}. > --- > configure.ac | 3 ++- > intel/intel_decode.c | 2 -- > 2 files changed, 2 insertions(+), 3 deletions(-) > >

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On Fri, Jan 15, 2016 at 12:12 PM, Emil Velikov wrote: > On 12 January 2016 at 23:14, Marek Olšák wrote: >> From: Marek Olšák >> >> It warns for all "{}" initializers. Well, I want us to use {}. >> --- >> configure.ac | 3 ++- >> intel/intel_decode.c | 2 -- > The whole of libdrm,

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

On 12 January 2016 at 23:14, Marek Olšák wrote: > From: Marek Olšák > > It warns for all "{}" initializers. Well, I want us to use {}. > --- > configure.ac | 3 ++- > intel/intel_decode.c | 2 -- The whole of libdrm, minus the intel_decode can get away without using such constructs.

[PATCH] configure.ac: disable annoying warning -Wmissing-field-initializers

From: Marek Olšák It warns for all "{}" initializers. Well, I want us to use {}. --- configure.ac | 3 ++- intel/intel_decode.c | 2 -- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/configure.ac b/configure.ac index c8c4ace..057a846 100644 ---