[PATCH - v3] readable error messages on non-ASCII systems

2010-05-21 Thread Greg Ames
forgot to cc: dev again. -- Forwarded message -- From: Greg Ames Date: Fri, May 21, 2010 at 10:28 AM Subject: Re: [PATCH - v3] readable error messages on non-ASCII systems To: Stefan Sperling On Thu, May 20, 2010 at 7:23 PM, Stefan Sperling wrote: > On Thu, May 20, 2010

Re: [PATCH - v3] readable error messages on non-ASCII systems

2010-05-20 Thread Stefan Sperling
On Fri, May 21, 2010 at 01:23:13AM +0200, Stefan Sperling wrote: > (I don't really see a need for svn to have its own ctype implementation), The need seems to be that we cannot rely on every locale correctly handling UTF-8 strings. But it seems APR would be a better home for this kind of thing. S

Re: [PATCH - v3] readable error messages on non-ASCII systems

2010-05-20 Thread Stefan Sperling
On Thu, May 20, 2010 at 03:10:04PM -0400, Greg Ames wrote: > I see that the svn ctype functions are hardwired to ASCII. If they called > the equivalent apr ctype functions instead, they should become portable to > EBCDIC. Doing that should change the behavior of > svn_utf__cstring_from_utf8_fuzzy

Re: [PATCH - v3] readable error messages on non-ASCII systems

2010-05-20 Thread Greg Ames
This isn't going to do the whole job, nor is patch #2. If I grep for 'svn_cmdline_printf.*\"' to find where native literal strings are passed to svn_cmdline_printf, I get 130 hits in trunk, for example: ./subversion/svn/util.c:SVN_ERR(svn_cmdline_printf(pool, _("\nWarning: %s\n"), ./subversio

[PATCH - v3] readable error messages on non-ASCII systems

2010-05-18 Thread Greg Ames
This version hits error.c only, no new APIs. Sending as an attachment per http://subversion.apache.org/docs/community-guide/general.html#patches [[[ Produce readable error messages on non-ASCII systems. * subversion/libsvn_subr/error.c: print_error(): convert err->message and the message