Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-28 Thread Martin Sebor
On 07/27/2017 01:13 PM, Richard Sandiford wrote: Martin Sebor writes: On 07/23/2017 02:42 PM, Volker Reichelt wrote: On 21 Jul, Martin Sebor wrote: On 07/20/2017 10:35 AM, Volker Reichelt wrote: Hi, the following patch introduces a new C++ warning option

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-27 Thread Richard Sandiford
Martin Sebor writes: > On 07/23/2017 02:42 PM, Volker Reichelt wrote: >> On 21 Jul, Martin Sebor wrote: >>> On 07/20/2017 10:35 AM, Volker Reichelt wrote: Hi, the following patch introduces a new C++ warning option -Wduplicated-access-specifiers that warns

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-25 Thread Volker Reichelt
On 23 Jul, Martin Sebor wrote: > On 07/23/2017 02:42 PM, Volker Reichelt wrote: >> On 21 Jul, Martin Sebor wrote: >>> On 07/20/2017 10:35 AM, Volker Reichelt wrote: Hi, the following patch introduces a new C++ warning option -Wduplicated-access-specifiers that warns about

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-23 Thread Martin Sebor
On 07/23/2017 02:42 PM, Volker Reichelt wrote: On 21 Jul, Martin Sebor wrote: On 07/20/2017 10:35 AM, Volker Reichelt wrote: Hi, the following patch introduces a new C++ warning option -Wduplicated-access-specifiers that warns about redundant access-specifiers in classes, e.g. class B {

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-23 Thread Volker Reichelt
On 21 Jul, David Malcolm wrote: > On Fri, 2017-07-21 at 19:58 +0200, Volker Reichelt wrote: >> On 21 Jul, David Malcolm wrote: >> > On Thu, 2017-07-20 at 18:35 +0200, Volker Reichelt wrote: >> >> Hi, >> >> >> >> the following patch introduces a new C++ warning option >> >>

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-23 Thread Volker Reichelt
On 21 Jul, Martin Sebor wrote: > On 07/20/2017 10:35 AM, Volker Reichelt wrote: >> Hi, >> >> the following patch introduces a new C++ warning option >> -Wduplicated-access-specifiers that warns about redundant >> access-specifiers in classes, e.g. >> >> class B >> { >> public: >>

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-21 Thread David Malcolm
On Fri, 2017-07-21 at 19:58 +0200, Volker Reichelt wrote: > On 21 Jul, David Malcolm wrote: > > On Thu, 2017-07-20 at 18:35 +0200, Volker Reichelt wrote: > >> Hi, > >> > >> the following patch introduces a new C++ warning option > >> -Wduplicated-access-specifiers that warns about redundant > >>

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-21 Thread Volker Reichelt
On 21 Jul, David Malcolm wrote: > On Thu, 2017-07-20 at 18:35 +0200, Volker Reichelt wrote: >> Hi, >> >> the following patch introduces a new C++ warning option >> -Wduplicated-access-specifiers that warns about redundant >> access-specifiers in classes, e.g. >> >> class B >> { >>

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-21 Thread Martin Sebor
On 07/20/2017 10:35 AM, Volker Reichelt wrote: Hi, the following patch introduces a new C++ warning option -Wduplicated-access-specifiers that warns about redundant access-specifiers in classes, e.g. class B { public: B(); private: void foo(); private: int i;

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-21 Thread Volker Reichelt
On 21 Jul, David Malcolm wrote: > On Fri, 2017-07-21 at 11:56 -0400, David Malcolm wrote: >> On Thu, 2017-07-20 at 18:35 +0200, Volker Reichelt wrote: >> > Hi, >> > >> > the following patch introduces a new C++ warning option >> > -Wduplicated-access-specifiers that warns about redundant >> >

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-21 Thread David Malcolm
On Fri, 2017-07-21 at 11:56 -0400, David Malcolm wrote: > On Thu, 2017-07-20 at 18:35 +0200, Volker Reichelt wrote: > > Hi, > > > > the following patch introduces a new C++ warning option > > -Wduplicated-access-specifiers that warns about redundant > > access-specifiers in classes, e.g. > > > >

Re: [PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-21 Thread David Malcolm
On Thu, 2017-07-20 at 18:35 +0200, Volker Reichelt wrote: > Hi, > > the following patch introduces a new C++ warning option > -Wduplicated-access-specifiers that warns about redundant > access-specifiers in classes, e.g. > > class B > { > public: > B(); > > private: >

[PATCH] New C++ warning option '-Wduplicated-access-specifiers'

2017-07-20 Thread Volker Reichelt
Hi, the following patch introduces a new C++ warning option -Wduplicated-access-specifiers that warns about redundant access-specifiers in classes, e.g. class B { public: B(); private: void foo(); private: int i; }; test.cc:8:5: warning: duplicate 'private'