Re: Patch - suspicious ownership, too wide scope

On 13.04.2011 11:38, Dmitry wrote:
> Hey.
>
> Attached is a patch that addresses suspiciuos stuff.
>
> First, an array is created with new[], then not released. Is its owndership passed somewhere? I've lloked into MSDN and didn't find any confirmation for that, still not sure, maybe I'm wrong.
>
> Then, an array is created with new[], then released with delete[] - there's a wrapper for that. Finally, some decalarations are suboptimally placed.

Ups, I rewrote that part before I checked your patch. Sorry.
But I'd appreciate it if you could review my changes.

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=2719541
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].