Re: Patch: auto_ptr is frequently abused in TSVN code

Hey.

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.

Best wishes.
Dmitry.

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

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