[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH 02/14] Fix character encoding aliases for OS/2

From: KO Myung-Hun
Subject: Re: [PATCH 02/14] Fix character encoding aliases for OS/2
Date: Fri, 19 Dec 2014 14:44:30 +0900
User-agent: Mozilla/5.0 (OS/2; Warp 4.5; rv:10.0.6esrpre) Gecko/20120715 Firefox/10.0.6esrpre SeaMonkey/2.7.2

KO Myung-Hun wrote:
> Hi/2.
> Daiki Ueno wrote:
>> Hello,
>> The change should work, but I have a feeling that something is wrong:
>>> +# if defined OS2
>>> +      /* To avoid the troubles of installing a separate file in the same
>>> +         directory as the DLL and of retrieving the DLL's directory at
>>> +         runtime, simply inline the aliases here.  */
>>> +
>>> +      cp = "bg_BG" "\0" "CP1251" "\0"
>> You are adding locale-to-charset mappings here, while all other mappings
>> defined are charset-to-charset.  Perhaps such a mappings could be
>> resolved in intl/localealias.c through the locale.alias file instead?
>> Your change might be necessary for other consumers than gettext.  If so,
>> could you explain the actual use-case in the ChangeLog and/or the above
>> comment?
> I'm using this in libiconv. On OS/2, a charset is not specified
> generally. Just only a locale is specified. And all LC_xxx are not
> specified. Just only LANG is specified. For examples, set LANG=ko_KR for
> Korean. And the case that charset aliases are used is that '.' to
> specify a charset is not found or a buffer is overflowed, which is
> impossible if a charset is correct. So charset-to-charset mapping is not
> useful on OS/2.
>> Also if it changes anything in the OS/2 port of gettext, consider
>> updating this file:
>> http://git.savannah.gnu.org/cgit/gettext.git/tree/os2/README.OS2
> Ok. If I work on gettext later, I'll consider.

I missed a patch.

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

Attachment: 0001-Fix-character-encoding-aliases-for-OS-2.patch
Description: Text document

reply via email to

[Prev in Thread] Current Thread [Next in Thread]