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

Re: svn commit: r10788 - in trunk/subversion: include libsvn_subr

From: Branko Čibej <brane_at_xbc.nu>
Date: 2004-09-01 02:15:16 CEST

lundblad@tigris.org wrote:

>Author: lundblad
>Date: Tue Aug 31 14:20:47 2004
>New Revision: 10788
>
>Modified:
> trunk/subversion/include/svn_utf.h
> trunk/subversion/libsvn_subr/cmdline.c
> trunk/subversion/libsvn_subr/utf.c
>Log:
>Fix issue #2016. Cache UTF8 translation handles in a global hash table
>instead of in the current pool, which improves performance, especially on
>Windows.
>
>
Apart from a bug that breaks compilation on (at least) FreeBSD, which I
suspect is caused by using "#ifdef APR_HAS_THREADS" in some places
instead of consistently testing with "#if"...

I think there is a better way to do this, that could avoid using global
data and locks: add the xlate cache to the svn_client_context_t struct,
and document that this structure is thread-specific. Most public
functions need to grow a thread-specific context parameter anyway,
although that will have to wait for 2.0. But I think it should be
possible to use the cache from the context in most places where we do
charset conversions now, by adding a set of conversion functions that
take a context parameter.

-- Brane

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Wed Sep 1 02:17:30 2004

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

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