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

Re: [TSVN] Another go at a patch for SVNFolderStatus

From: SteveKing <steveking_at_gmx.ch>
Date: 2004-10-28 19:18:20 CEST

Will Dean wrote:

> Let's not do that then, let's do what it asks for.

That's what I'm trying to do ;)

> There is definitely a bug in there are the moment, which is the following:
> The OS asks for
> \will\tsvntest\top\sub1
> (which is a folder)
> TSVN then mistakes that for a file (by falling out of the if(isFolder)
> clause, chops the end off and
> asks SVN for the status of the folder:
> \will\tsvntest\top

And that's usually correct. Imagine the following:

now, the shell asks for the status of

if we then get the status for
we will fetch the status for _all_ the folders and files in that folder,
which includes sub1, sub2, file1 and file2 in one step. That's what this
is about.
The problem here is that sometimes, the column provider is the first to
ask for
even if your showing only the contents of that folder.

> Ignoring anything to do with SVN bugs, surely this wasn't what you
> intended?

It was. I just didn't take the column provider into the calculation ;)

> What I'm seeing is that *nobody* asks for the status of the parent
> folder, but TSVN mistakenly chops down the path and asks SVN for it.

Yes, the shell asks for the status of the parent via the column provider.

> I think if we just looked up the item we were asked for that would be a
> good start!

definitely not! Try it once - you'll revert your changes after the first
try ;)

I'm currently trying to differ between the column provider calls and the
icon overlay calls. I think if we can assume that the shell won't call
the column provider for the parent folder somewhere 'in between' the
requests for the other items it will work. I'll do some tests and if all
goes well I'll commit them.


   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest Interface to (Sub)Version Control
    /_/   \_\     http://tortoisesvn.tigris.org
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Thu Oct 28 21:23:45 2004

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.