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

Re: [TSVN] Possible bug in

From: SteveKing <steveking_at_gmx.ch>
Date: 2004-12-16 11:16:07 CET

Will Dean wrote:

> CSVNStatusListCtrl::GetStatus
>
> Contains the following code, around line 274:
>
> // check if the unversioned folder is maybe versioned. This
> // could happen with nested layouts
> if (SVNStatus::GetAllStatus(temp) != svn_wc_status_unversioned)
> continue; //ignore nested layouts
>
> I don't believe that 'temp' is assigned to anything at this point - can
> you check this?

You're right. Fixed in revision 2133 (together with your perf patch).

> I am in the process of trying to reduce the number of deep string copies
> which are made for each file (currently about 6.5 in my test case) and
> as part of that, I am trying to make sense of all the strings in that
> method which are called things like 'temp', 't', 'filename', 'temppath',
> etc.
>
> (In fact, more than making sense of them, I'm going to try and get rid
> of them!)
>
> GetStatus is now an incredibly complicated function, and I think there
> is a lot of unnecessary copying of stuff going on - do you mind if I try
> and refactor it somewhat?

Sure! If you can make sense of that code, I'd be happy to apply any
patch you send me ;)

But be carefull - that part of code isn't complicated without reason.

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
Received on Thu Dec 16 11:18:39 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.