Re: Patch - too long function, duplicate code, unnecessary actions.

On 05.03.2011 11:06, Dmitry wrote:
> Hey.
>
> Attached is an untested patch that addresses the following minor problems is RemoteCacheLink:
> - a function is too long
> - no sense to first have "ifndef", then "else" - it should be "ifdef", then "else"
> - if the process is 32-bit and not Wow64 and the first attemp to start TSVNCache.exe would fail the second attempt would have been made for the very same path
> - if the process is 32-bit Wow64 and there's a path from which a 64-bit TSVNCache would start successfully the path to 32-bit version would be computed and never used
> - code for computing the path to TSVNCache.exe is duplicated

Committed in r20970.

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