[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: Patch - unnecessary actions

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Fri, 08 Jul 2011 18:35:13 +0200

On 08.07.2011 16:40, Dmitry wrote:
> Hey.
>
> Attached is an untested patch that addresses unnecessary actions.
> Sometimes it's a cutoff check done later than it could be. Sometimes
> it's doing an action and then conditionally overriding it. Sometimes
> it is checking for some condition and then checking for a mutually
> exclusive one immediately.

Committed in r21669.
One small change: I reverted one change you made since it was wrong (you
forgot to set the status in some situations) and I don't see the benefit
of that change.
But only that one change.

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=2788115
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2011-07-08 18:35:25 CEST

This is an archived mail posted to the TortoiseSVN Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.