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

Re: Patch - wrong casts, no cleanup in destructor

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Mon, 07 Feb 2011 19:49:21 +0100

On 07.02.2011 07:58, Dmitry wrote:
> Hey.
>
>>> That code is copied from CShellExt in TortoiseShell, where the class
>>> is indeed inherited from all those interfaces. static_cast works
>>> there - I copied that code into a separate project to test. If
>>> static_cast was used initially the compiler would prevent such
>>> unfortunate use of copy-paste.
>> Ups. You're absolutely right.
>> Fixed in r20823.
>
> Attached is an untested patch with more of the same in all other COM servers.

Committed in r20823.

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=2702691
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-02-07 19:49:36 CET

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.