Re: stdcxx issue 1058

2012-09-02 Thread Wojciech Meyer
Hi Stefan,

Stefan Teleman stefan.tele...@gmail.com writes:

 On Fri, Aug 31, 2012 at 1:29 PM, Liviu Nicoara nikko...@hates.ms wrote:
 On 08/31/12 13:14, Stefan Teleman wrote:


 In June this year I committed r1353821 to trunk which fixes stdcxx-1058.

 I have the patches for 1058 ready to commit to branches (4.2.x and 4.3.x).

 OK to go?

 The patch looks ok to me. What seems to be the problem?  +1

I also had a initial look at the patch. Looks OK, I looked at the other
occurrences of operator delete to see if we are not doing the same
thing.

--
Wojciech Meyer
http://danmey.org


Re: stdcxx issue 1058

2012-08-31 Thread Liviu Nicoara

On 08/31/12 13:14, Stefan Teleman wrote:


In June this year I committed r1353821 to trunk which fixes stdcxx-1058.

I have the patches for 1058 ready to commit to branches (4.2.x and 4.3.x).

OK to go?


The patch looks ok to me. What seems to be the problem?  +1

L


Re: stdcxx issue 1058

2012-08-31 Thread Stefan Teleman
On Fri, Aug 31, 2012 at 1:29 PM, Liviu Nicoara nikko...@hates.ms wrote:
 On 08/31/12 13:14, Stefan Teleman wrote:


 In June this year I committed r1353821 to trunk which fixes stdcxx-1058.

 I have the patches for 1058 ready to commit to branches (4.2.x and 4.3.x).

 OK to go?

 The patch looks ok to me. What seems to be the problem?  +1

Done.

trunk revision 1353821
branches/4.2.x revision 1379523
branches/4.3.x revision 1379520

--Stefan

-- 
Stefan Teleman
KDE e.V.
stefan.tele...@gmail.com