Re: svn commit: r11783 - trunk/subversion/svnserve

Branko Čibej wrote:
> Julian Foad wrote:
>
>>>> apr_socket_listen(sock, 7);
>>>> +
>>>> + /* svn_cmdline_init() sets up the locale, but when we serve clients, we
>>>> + always want the "C" locale. */
>>>> + if (!setlocale (LC_ALL, "C"))
>>>> + {
>>>> + fprintf(stderr, "Can't set locale back to the \"C\" locale\n");
>>
>> Can that call actually fail?
>
> Any call can fail.

If you mean to imply that, therefore, every call should be checked, then should
not the above-quoted "apr_socket_listen" and "fprintf" be checked too?

Can setlocale(LC_ALL, "C") realistically be anticipated to fail on systems that
svnserve can run on, and does failure have such consequences that it is worth
worrying about and checking for?

- Julian

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Nov 9 16:50:41 2004