Re: [PATCH][OBVIOUS] Remove extra line in MAINTAINERS.

2018-08-10 Thread Martin Liška
On 08/10/2018 01:12 PM, Paolo Carlini wrote:
> Hi,
> 
> On 10/08/2018 12:44, Martin Liška wrote:
>> On 08/10/2018 12:38 PM, Paolo Carlini wrote:
>>> Hi,
>>>
>>> On 10/08/2018 12:35, Martin Liška wrote:
 Hi.

 This removes one extra line that's a typo.
>>> It's not ;) The complete line is "c++ runtime libs special modes". See? 
>>> Admittedly, we may want to have something clearer, but just removing that 
>>> line isn't the way to go.
>>>
>>> Paolo.
>> Oup, I was too eager.
>> So then what about putting "c++ runtime libs special modes" on one line?
> Yes, that would be nice, but I think Francois didn't manage to do that while 
> keeping a consistent formatting, that line is *long*!
> 
> Paolo.

Then I'm going to install following patch.

Martin
>From c72c46fce810b63e162ebcce8abc2fd1d44e2181 Mon Sep 17 00:00:00 2001
From: marxin 
Date: Fri, 10 Aug 2018 13:36:53 +0200
Subject: [PATCH] Fix wrongly removed line.

ChangeLog:

2018-08-10  Martin Liska  

	* MAINTAINERS: Revert change in previous commit and
join lines.
---
 MAINTAINERS | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/MAINTAINERS b/MAINTAINERS
index 8d42c91b2d7..a9f20d73afe 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -188,7 +188,7 @@ c++ runtime libs	Paolo Carlini		
 c++ runtime libs	Ulrich Drepper		
 c++ runtime libs	Benjamin De Kosnik	
 c++ runtime libs	Jonathan Wakely		
-special modes		François Dumont		
+c++ runtime libs special modes		François Dumont		
 fixincludes		Bruce Korb		
 *gimpl*			Jakub Jelinek		
 *gimpl*			Aldy Hernandez		
-- 
2.18.0



Re: [PATCH][OBVIOUS] Remove extra line in MAINTAINERS.

2018-08-10 Thread Paolo Carlini

Hi,

On 10/08/2018 12:44, Martin Liška wrote:

On 08/10/2018 12:38 PM, Paolo Carlini wrote:

Hi,

On 10/08/2018 12:35, Martin Liška wrote:

Hi.

This removes one extra line that's a typo.

It's not ;) The complete line is "c++ runtime libs special modes". See? 
Admittedly, we may want to have something clearer, but just removing that line isn't the 
way to go.

Paolo.

Oup, I was too eager.
So then what about putting "c++ runtime libs special modes" on one line?
Yes, that would be nice, but I think Francois didn't manage to do that 
while keeping a consistent formatting, that line is *long*!


Paolo.


Re: [PATCH][OBVIOUS] Remove extra line in MAINTAINERS.

2018-08-10 Thread Martin Liška
On 08/10/2018 12:38 PM, Paolo Carlini wrote:
> Hi,
> 
> On 10/08/2018 12:35, Martin Liška wrote:
>> Hi.
>>
>> This removes one extra line that's a typo.
> It's not ;) The complete line is "c++ runtime libs special modes". See? 
> Admittedly, we may want to have something clearer, but just removing that 
> line isn't the way to go.
> 
> Paolo.

Oup, I was too eager.
So then what about putting "c++ runtime libs special modes" on one line?

Martin


Re: [PATCH][OBVIOUS] Remove extra line in MAINTAINERS.

2018-08-10 Thread Paolo Carlini

Hi,

On 10/08/2018 12:35, Martin Liška wrote:

Hi.

This removes one extra line that's a typo.
It's not ;) The complete line is "c++ runtime libs special modes". See? 
Admittedly, we may want to have something clearer, but just removing 
that line isn't the way to go.


Paolo.


[PATCH][OBVIOUS] Remove extra line in MAINTAINERS.

2018-08-10 Thread Martin Liška
Hi.

This removes one extra line that's a typo.

Martin

ChangeLog:

2018-08-10  Martin Liska  

* MAINTAINERS: Remove extra line.
---
 MAINTAINERS | 1 -
 1 file changed, 1 deletion(-)


diff --git a/MAINTAINERS b/MAINTAINERS
index f96ab622146..8d42c91b2d7 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -188,7 +188,6 @@ c++ runtime libs	Paolo Carlini		
 c++ runtime libs	Ulrich Drepper		
 c++ runtime libs	Benjamin De Kosnik	
 c++ runtime libs	Jonathan Wakely		
-c++ runtime libs
 special modes		François Dumont		
 fixincludes		Bruce Korb		
 *gimpl*			Jakub Jelinek