[vlc-devel] [PATCH] demux: hls: playlist: fall back to "C" locale

[KO Myung-Hun]

Hi/2.

Francois Cartegnie wrote:
> Le 06/12/2015 10:58, KO Myung-Hun a écrit :
>> Some systems such as OS/2, do not support other locales than C and
>> POSIX. And those locales cause SIGABRT due to std::runtime_error
>> exception, especially, when playing youtube live streaming.
> 
> 
>>      std::istringstream is(value);
>> -    is.imbue(std::locale(""));
>> +    try
>> +    {
>> +        is.imbue(std::locale(""));
>> +    }
>> +    catch(const std::runtime_error &e)
>> +    {
>> +        is.imbue(std::locale("C"));
>> +    }
> 
> 
> The std::locale("") was wrong.
> The correct fix is to always convert using C locale.
> 

Updated.

-- 
KO Myung-Hun

Using Mozilla SeaMonkey 2.7.2
Under OS/2 Warp 4 for Korean with FixPak #15
In VirtualBox v4.1.32 on Intel Core i7-3615QM 2.30GHz with 8GB RAM

Korean OS/2 User Community : http://www.ecomstation.co.kr

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-demux-hls-playlist-use-C-locale.patch
Type: application/x-patch
Size: 1102 bytes
Desc: not available
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20151211/a4e1212e/attachment.bin>