Angus Leeming wrote:
> Peter Kümmel <[EMAIL PROTECTED]> writes:
>> +#ifdef HAVE_LC_MESSAGES
>> +        lc_msgs = setlocale(LC_MESSAGES, NULL);
>> +#endif
> 
> Good detective work (again!) but there's no need to define a macro to use a
> macro. Use #ifdef LC_MESSAGES instead.

The problem is that currently there IS a LC_MESSAGES, defined by the intl 
header.
So we can't check if it's not defined by the msvc headers.


> What happens if you define LC_MESSAGES to 1729 as you showed us in that 
> Solaris
> block?

This is done by the svn version and 1729 leads to the crash.

Without touching the intl files, we must use HAVE_LC_MESSAGES
(there was already a HAVE_LC_MESSAGES in the code),
test LC_MESSAGE for 1729, or disable the code with _WIN32.

But is it important not to touch the intl lib?

Peter

Reply via email to