Re: svn commit: r9714 - trunk/subversion/libsvn_wc

On Fri 2004-05-14 at 23:40:49 +0200, Tobias Ringström wrote:
> Ben Collins-Sussman wrote:
> >On Fri, 2004-05-14 at 16:09, bliss@tigris.org wrote:
> >
> >>* subversion/libsvn_wc/adm_crawler.c
> >> (report_revisions): Call svn_io_check_path for each entry instead of
> >> calling svn_io_get_dirents and comparing strings.
> >
> >Um... isn't this going to cause a *huge* slowdown when the 'svn commit'
> >scans for modified files?

I wondered the same and was just about to write an email...

> I thought that svn_io_get_dirents already called stat to get the type
> and that it would be the same amount of work. On a closer look (and to
> my big surprise) there seems to be support for a d_type entry in the
> dirent on some systems to avoid having to call stat. I'll continue
> investigating and find another solution if neccessary.

I didn't too deep into it, but it looks like one could keep the
previous logic and only add the svn_io_check_path call if it "looks"
like the item is missing in order to be sure the file system "thinks"
the same.

Bye,

        Benjamin.

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri May 14 23:50:59 2004