Re: Patch: reducing amount of duplicate code

Okay, I've factored out the code for FormatMessage(). Haven't tested it.

As for creating .cpp files just to prevent inlining the code of trivial helper classes I seriously don't understand the intent behind that. Inlining is not an inherent evil by itself and introducing an extra file that will consist mainly of the GPL header doesn't look tempting at all.

Moving error handling to one central place is not that obvious. In some cases CreateProcess() errors are silently ignored, in some cases a fixed error message is shown and in some cases it's a custom message - the logic is different each time and factoring it out is not trivial.

Best wishes.
Dmitry.

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

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