X-Recipient: archive-cygwin@delorie.com
X-SWARE-Spam-Status: No, hits=-1.9 required=5.0 	tests=AWL,BAYES_00,SARE_MSGID_LONG40,SPF_PASS
X-Spam-Check-By: sourceware.org
MIME-Version: 1.0
In-Reply-To: <20090821210536.GM32408@calimero.vinschen.de>
References: <416096c60908201343g6134c93ao3f4646f6e3fc0dfe@mail.gmail.com> 	 <20090821074927.GD32408@calimero.vinschen.de> 	 <20090821210536.GM32408@calimero.vinschen.de>
Date: Sat, 22 Aug 2009 09:45:40 +0100
Message-ID: <416096c60908220145q1b186194l67c279fd394ec96a@mail.gmail.com>
Subject: Re: KOI8
From: Andy Koppe <andy.koppe@gmail.com>
To: cygwin@cygwin.com
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
X-IsSubscribed: yes
Mailing-List: contact cygwin-help@cygwin.com; run by ezmlm
Precedence: bulk
List-Id: <cygwin.cygwin.com>
List-Unsubscribe: <mailto:cygwin-unsubscribe-archive-cygwin=delorie.com@cygwin.com>
List-Subscribe: <mailto:cygwin-subscribe@cygwin.com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin@cygwin.com>
List-Help: <mailto:cygwin-help@cygwin.com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner@cygwin.com
Mail-Followup-To: cygwin@cygwin.com
Delivered-To: mailing list cygwin@cygwin.com

Corinna Vinschen:
>> > Anyway, to help support them, the attached patch adds the KOI8
>> > charsets to newlib's Unicode conversion and ctype tables. I took the
>> > conversion tables from iconv and adapted the ctype tables from the
>> > CP1251 version. Since KOI8 has printable characters in the C1 range
>> > from 0x80 to 0x9F, it seems easiest to treat them as Windows
>> > codepages.
>> >
>> > To complete support, "KOI8-R" and "KOI8-U" would need to be recognised
>> > in _setlocale_r and mapped to codepages 20866 and 21866.
>>
>> I'd suggest to add the missing code to loadlocale() =C2=A0(the internally
>> used charset should be set to "CP20866"/"CP21866", but it seems you know
>> this already) and send the entire patch, together with a ChangeLog
>> entry, to the newlib list. =C2=A0If you could base it on my pending prop=
osal
>> to make the charset case insensitive
>> http://sourceware.org/ml/newlib/2009/msg00840.html, that would be great.
>
> I applied this patch =C2=A0a couple of minutes ago, so you can simply base
> your patch on current CVS.

Okay, will do.

Andy

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple

