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

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

Best wishes.
Dmitry.

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

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