[issue4474] PyUnicode_FromWideChar incorrect for characters outside the BMP (unix only)

[Marc-Andre Lemburg]

Marc-Andre Lemburg <mal at egenix.com> added the comment:

On 2009-02-24 20:39, Mark Dickinson wrote:
> Mark Dickinson <dickinsm at gmail.com> added the comment:
> 
> Updated Victor's patch:
> 
>   - applies cleanly against newly whitespace-normalized unicodeobject.c
>   - renamed USE_WCHAR_SURROGATE to CONVERT_WCHAR_TO_SURROGATES
>   - add defined(SIZEOF_WCHAR_T) check
> 
> I find the patched version of PyUnicode_FromWideChar quite hard to follow 
> with all the #ifdefery.  I wonder whether it might be better to define a 
> _PyUnicode16_FromWideChar32 helper function instead.

Same here. It would be better to have a single #ifdef #else #endif
block with one branch for the CONVERT_WCHAR_TO_SURROGATES case and the
other for the normal operation.

No need for a new helper function.

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue4474>
_______________________________________