Re: [PATCH 3/3] fhandler_pty_slave::setup_locale: respect charset == "UTF-8"

2020-09-06 Thread Takashi Yano via Cygwin-patches
On Mon, 7 Sep 2020 01:04:13 +0900 Takashi Yano via Cygwin-patches wrote: > On Sun, 6 Sep 2020 19:15:30 +0900 > Takashi Yano via Cygwin-patches wrote: > > On Sun, 6 Sep 2020 17:57:03 +0900 > > Takashi Yano via Cygwin-patches wrote: > > > On Sat, 5 Sep 2020 23:15:16 +0900 > > > Takashi Yano via Cy

Re: [PATCH 3/3] fhandler_pty_slave::setup_locale: respect charset == "UTF-8"

2020-09-06 Thread Takashi Yano via Cygwin-patches
On Sun, 6 Sep 2020 19:15:30 +0900 Takashi Yano via Cygwin-patches wrote: > On Sun, 6 Sep 2020 17:57:03 +0900 > Takashi Yano via Cygwin-patches wrote: > > On Sat, 5 Sep 2020 23:15:16 +0900 > > Takashi Yano via Cygwin-patches wrote: > > > Hi Corinna, > > > > > > On Sat, 5 Sep 2020 20:15:06 +0900

Re: [PATCH 3/3] fhandler_pty_slave::setup_locale: respect charset == "UTF-8"

2020-09-06 Thread Takashi Yano via Cygwin-patches
Hi Corinna, On Fri, 4 Sep 2020 14:44:00 +0200 Corinna Vinschen wrote: > +case 'I': /* ISO-8859-x */ > + codepage = strtoul (charset + 9, NULL, 10); > + break; This should be: codepage = strtoul (charset + 9, NULL, 10) + 28590; shouldn't it? -- Takashi Yano

Re: [PATCH 3/3] fhandler_pty_slave::setup_locale: respect charset == "UTF-8"

2020-09-06 Thread Takashi Yano via Cygwin-patches
On Sun, 6 Sep 2020 17:57:03 +0900 Takashi Yano via Cygwin-patches wrote: > On Sat, 5 Sep 2020 23:15:16 +0900 > Takashi Yano via Cygwin-patches wrote: > > Hi Corinna, > > > > On Sat, 5 Sep 2020 20:15:06 +0900 > > Takashi Yano via Cygwin-patches wrote: > > > On Sat, 5 Sep 2020 17:43:01 +0900 > >

Re: [PATCH 3/3] fhandler_pty_slave::setup_locale: respect charset == "UTF-8"

2020-09-06 Thread Takashi Yano via Cygwin-patches
On Sat, 5 Sep 2020 23:15:16 +0900 Takashi Yano via Cygwin-patches wrote: > Hi Corinna, > > On Sat, 5 Sep 2020 20:15:06 +0900 > Takashi Yano via Cygwin-patches wrote: > > On Sat, 5 Sep 2020 17:43:01 +0900 > > Takashi Yano via Cygwin-patches wrote: > > > Hi Corinna, > > > > > > On Fri, 4 Sep 202