[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: r21936 is damn suspicious

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Mon, 05 Sep 2011 20:20:18 +0200

On 05.09.2011 14:37, Dmitry wrote:
> Hey.
>
> May I wonder why r21936 was committed?

To reduce the memory allocated on the stack by allocating the memory on
the heap instead.

> Currently it uses raw pointers and this is not exception-safe. Also

I don't quite see the problem here?

> in TortoiseIDiff.cpp if control flows through "return (int)
> msg.wParam;" the window object is not deleted.

Fixed in r1943.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2835656
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-09-05 20:20:23 CEST

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.