Re: Patch: auto_ptr is frequently abused in TSVN code

Dmitry <wipedout_at_yandex.ru> wrote:

> Here's another patch using auto_buffer for array lifetime management. It
also addresses unused
> local CString variables found every here and there, redundant nested
code blocks and rewrites long
> methods for less block nesting and reduces code duplication.
>
> The patch is untested and needs a review.
>
> Also in SetColorPage.cpp there is (and was) a local variable
"adjustedcolor" that is not
> initialized and passed as an in parameter to a function. That's
undefined behaviour. I have no
> idea of what correct code should be at that place.

Commencing review ...

-- Stefan^2.

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

To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2009-08-31 13:20:25 CEST