[prev in list] [next in list] [prev in thread] [next in thread] 

List:       tortoisesvn-dev
Subject:    Re: [TSVN] Another go at a patch for SVNFolderStatus
From:       SteveKing <steveking () gmx ! ch>
Date:       2004-10-29 21:12:59
Message-ID: 4182B25B.2040801 () gmx ! ch
[Download RAW message or body]

Will Dean wrote:
> At 22:49 29/10/2004 +0200, you wrote:
> 
>> /me thinking that you're right.
> 
> 
> Blimey.  Have you been drinking?

You might not believe this, but I don't drink. I just don't like the 
taste of alcohol...

>> I'll have a look at that this weekend. We have to be very careful with 
>> '\' and '/' because Windows likes '\', but Subversion _only_ knows '/' 
>> (Windows API's usually can deal with both, not all though).
> 
> 
> I'm interested in which Windows APIs can't cope with / - as far as I 
> knew, / has been OK in file paths since about DOS2.   (But not on 
> command lines, obviously.)

That's what I was thinking too. But try once the shell API 
SHFileOperation() - it will work somehow, but not everything will work 
as expected.

> But you're right, you have to be very careful where there's lots of 
> string comparison going on.  My version here is running with a \ cache, 
> but I did get caught by a couple of things (the strchr in BuildCache, 
> for example)

That's why I don't just want to jump in there and start changing code. I 
first want to test the whole changes - especially for the shell 
extension part. A crash in the main part isn't that bad, but if the 
shell extension crashes, the whole desktop process goes down with it!

Stefan

-- 
        ___
   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





[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic