Use ordinary C strings for filenames?

While I'm at it...

Since valid names in a Subversion filesystem may not contain the null
character, it's not *necessary* to use svn_string_t for filename
arguments to FS functions. APR, together with ANSI, provide a full
set of primitives for dealing with standard null-terminated strings.
And debuggers print them nicely.

I never mutate a string the user passes in, so the fact that an
svn_string_t knows its pool doesn't matter.

Should I use `[const] char *' for name/path arguments to filesystem
functions, instead of `svn_string_t'?

I don't think the change will take much work.
Received on Sat Oct 21 14:36:14 2006