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

Re: svn commit: rev 1563 - trunk/subversion/include trunk/subversion/libsvn_subr trunk/subversion/clients/cmdline trunk/subversion/libsvn_ra_dav

From: Greg Stein <gstein_at_lyra.org>
Date: 2002-03-21 00:12:43 CET

On Wed, Mar 20, 2002 at 03:52:20PM -0600, Karl Fogel wrote:
> Branko Cibej <brane@xbc.nu> writes:
> > I seem to remember that string_t was created to make swig bindings
> > easier to write. If that's not the case any more, then I really mind
> > changing this.
>
> Mmmm... Could be. I can't remember if it helps SWIG bindings or not.
> Greg?

svn_stringbuf_t is a pain for *any* language bindings (even the explicit
ones like the hand-coded Ruby and Java bindings). The problem is that a
pool is required to turn a native-string into a stringbuf. But it can be
hard to find "the right pool" to effect that conversion. Thus, elimination
of *stringbuf* was desired.

Part of that elimination was establishing svn_string_t, and part was
switching over to 'const char *' where appropriate.

IOW, svn_string_t is totally fine for language bindings.

(and note that the 'const char *' types are fine, too; it is also important
 that the svn_string_t refers to a 'const char *' for both inherent utility,
 and for the bindings)

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Thu Mar 21 00:09:53 2002

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.