Patch - suspicious ownership, too wide scope

From: Dmitry <wipedout_at_yandex.ru>
Date: Wed, 13 Apr 2011 13:38:04 +0400

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.

Best wishes.
Dmitry.

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=2719372

To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].

application/octet-stream attachment: SuspiciousOwnershipSuboptimalDeclarations.patch

Received on 2011-04-13 11:38:09 CEST

This message: [ Message body ]
Next message: Stefan Küng: "Re: encrypting credentials is done plain wrong"
Previous message: Dmitry: "encrypting credentials is done plain wrong"
Next in thread: Stefan Küng: "Re: Patch - suspicious ownership, too wide scope"
Reply: Stefan Küng: "Re: Patch - suspicious ownership, too wide scope"

Contemporary messages sorted: [ by date ] [ by thread ] [ by subject ] [ by author ] [ by messages with attachments ]