iswcntrl() regression

iswcntrl() regression

[Bruno Haible]

Hi,

POSIX [1] requires that iswcntrl (EOF) == 0.

This was true in         CYGWIN_NT-10.0 2.9.0(0.318/5/3) (x86_64)
but is no longer true in CYGWIN_NT-10.0 2.11.1(0.329/5/3) (x86_64).
(Reported by Assaf Gordon.)

Looking at the changes that occurred in the cygwin git between
2.9.0 (on 2017-08-03) and 2.11.1 (on 2018-08-31)
it appears to me that the culprit is the commit
41f72ab4d7c404b8ac2a5e1187d79164992e4483 (on 2018-03-09).

I would like to CC its author, Thomas Wolff, but he gave an invalid
email address <mintty@users.noreply.github.com> in the commit.

This regression was found by gnulib's test suite
https://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=blob;f=tests/test-wctype-h.c

Bruno


--
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

Re: iswcntrl() regression

[Thomas Wolff]

On 19.12.2018 13:53, Bruno Haible wrote:
> Hi,
>
> POSIX [1] requires that iswcntrl (EOF) == 0.
Interesting special case, as EOF is not a character.
In newlib/libc/ctype, this could be fixed in iswcntrl_l.c as a special 
condition, or injected as a special case in the categories.t table.
Are there any other special requirements concerning EOF?
Thomas

>
> This was true in         CYGWIN_NT-10.0 2.9.0(0.318/5/3) (x86_64)
> but is no longer true in CYGWIN_NT-10.0 2.11.1(0.329/5/3) (x86_64).
> (Reported by Assaf Gordon.)
>
> Looking at the changes that occurred in the cygwin git between
> 2.9.0 (on 2017-08-03) and 2.11.1 (on 2018-08-31)
> it appears to me that the culprit is the commit
> 41f72ab4d7c404b8ac2a5e1187d79164992e4483 (on 2018-03-09).
>
> I would like to CC its author, Thomas Wolff, but he gave an invalid
> email address <mintty@users.noreply.github.com> in the commit.
>
> This regression was found by gnulib's test suite
> https://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=blob;f=tests/test-wctype-h.c
>
> Bruno
>
>
> --
> 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
>


--
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

Re: iswcntrl() regression

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 1324 bytes --]

CCed Bruno and Assaf, I guess both are not subscribed to the Cygwin ML.

On Dec 19 14:25, Thomas Wolff wrote:
> On 19.12.2018 13:53, Bruno Haible wrote:
> > Hi,
> > 
> > POSIX [1] requires that iswcntrl (EOF) == 0.
> Interesting special case, as EOF is not a character.
> In newlib/libc/ctype, this could be fixed in iswcntrl_l.c as a special
> condition, or injected as a special case in the categories.t table.
> Are there any other special requirements concerning EOF?
> Thomas
> 
> > 
> > This was true in         CYGWIN_NT-10.0 2.9.0(0.318/5/3) (x86_64)
> > but is no longer true in CYGWIN_NT-10.0 2.11.1(0.329/5/3) (x86_64).
> > (Reported by Assaf Gordon.)
> > 
> > Looking at the changes that occurred in the cygwin git between
> > 2.9.0 (on 2017-08-03) and 2.11.1 (on 2018-08-31)
> > it appears to me that the culprit is the commit
> > 41f72ab4d7c404b8ac2a5e1187d79164992e4483 (on 2018-03-09).
> > 
> > I would like to CC its author, Thomas Wolff, but he gave an invalid
> > email address <mintty@users.noreply.github.com> in the commit.
> > 
> > This regression was found by gnulib's test suite
> > https://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=blob;f=tests/test-wctype-h.c
> > 
> > Bruno

Thanks for the report, Bruno!


Corinna

-- 
Corinna Vinschen
Cygwin Maintainer

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: iswcntrl() regression

[Bruno Haible]

Hello Thomas,

Oops 1: I forgot to add the reference to POSIX:
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html

Oops 2:
The relevant expression is iswcntrl (WEOF), not iswcntrl (EOF).

> > Are there any other special requirements concerning EOF?

WEOF is a special value. The difference between wint_t and wchar_t is
essentially that WEOF fits into wint_t but is not guaranteed to fit in wchar_t.
(Like EOF fits into 'int' but usually does not fit in 'unsigned char'.)

WEOF is special for all functions declared in <wctype.h>, see

http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswalnum.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswalpha.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswblank.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswdigit.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswgraph.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswlower.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswprint.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswpunct.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswspace.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswupper.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswxdigit.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswctype.html

http://pubs.opengroup.org/onlinepubs/9699919799/functions/towlower.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/towupper.html
http://pubs.opengroup.org/onlinepubs/9699919799/functions/towctrans.html

> this could be fixed in iswcntrl_l.c as a special condition, or
> injected as a special case in the categories.t table.

Probably, yes.

Bruno


--
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

Re: iswcntrl() regression

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 2125 bytes --]

On Dec 19 21:57, Bruno Haible wrote:
> Hello Thomas,
> 
> Oops 1: I forgot to add the reference to POSIX:
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html
> 
> Oops 2:
> The relevant expression is iswcntrl (WEOF), not iswcntrl (EOF).
> 
> > > Are there any other special requirements concerning EOF?
> 
> WEOF is a special value. The difference between wint_t and wchar_t is
> essentially that WEOF fits into wint_t but is not guaranteed to fit in wchar_t.
> (Like EOF fits into 'int' but usually does not fit in 'unsigned char'.)
> 
> WEOF is special for all functions declared in <wctype.h>, see
> 
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswalnum.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswalpha.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswblank.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswdigit.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswgraph.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswlower.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswprint.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswpunct.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswspace.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswupper.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswxdigit.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswctype.html
> 
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/towlower.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/towupper.html
> http://pubs.opengroup.org/onlinepubs/9699919799/functions/towctrans.html
> 
> > this could be fixed in iswcntrl_l.c as a special condition, or
> > injected as a special case in the categories.t table.
> 
> Probably, yes.

Thomas, any input?  Are you going to provide patches?


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: iswcntrl() regression

[Thomas Wolff]

Am 06.01.2019 um 21:21 schrieb Corinna Vinschen:
> On Dec 19 21:57, Bruno Haible wrote:
>> Hello Thomas,
>>
>> Oops 1: I forgot to add the reference to POSIX:
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html
>>
>> Oops 2:
>> The relevant expression is iswcntrl (WEOF), not iswcntrl (EOF).
>>
>>>> Are there any other special requirements concerning EOF?
>> WEOF is a special value. The difference between wint_t and wchar_t is
>> essentially that WEOF fits into wint_t but is not guaranteed to fit in wchar_t.
>> (Like EOF fits into 'int' but usually does not fit in 'unsigned char'.)
>>
>> WEOF is special for all functions declared in <wctype.h>, see
>>
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswalnum.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswalpha.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswblank.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswdigit.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswgraph.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswlower.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswprint.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswpunct.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswspace.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswupper.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswxdigit.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswctype.html
>>
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/towlower.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/towupper.html
>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/towctrans.html
>>
>>> this could be fixed in iswcntrl_l.c as a special condition, or
>>> injected as a special case in the categories.t table.
>> Probably, yes.
> Thomas, any input?  Are you going to provide patches?
Well, yes, now I'm assuming that this is the only WEOF special case and 
I'll look after it this week.
Thomas

--
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

RE: iswcntrl() regression

[Wong, Samuel]

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset="utf-8", Size: 2279 bytes --]

Hi All,

I have tried to unsubscribe from the Cygwin mailing list.

I have tried simple way to unsubscribe by entering Mailing list name, Your email address.  But I cannot find out the mailing list name.
I have read the unsubscribe full, but still cannot find the mailing list name.

Please help to unsubscribe me for all mailing lists with Cygwin.

My email address is samuel.wong@intel.com



Thanks
Sam.

-----Original Message-----
From: cygwin-owner@cygwin.com <cygwin-owner@cygwin.com> On Behalf Of Corinna Vinschen
Sent: Wednesday, December 19, 2018 5:37 AM
To: cygwin@cygwin.com
Cc: Bruno Haible <bruno@clisp.org>; Assaf Gordon <assafgordon@gmail.com>
Subject: Re: iswcntrl() regression

CCed Bruno and Assaf, I guess both are not subscribed to the Cygwin ML.

On Dec 19 14:25, Thomas Wolff wrote:
> On 19.12.2018 13:53, Bruno Haible wrote:
> > Hi,
> > 
> > POSIX [1] requires that iswcntrl (EOF) == 0.
> Interesting special case, as EOF is not a character.
> In newlib/libc/ctype, this could be fixed in iswcntrl_l.c as a special 
> condition, or injected as a special case in the categories.t table.
> Are there any other special requirements concerning EOF?
> Thomas
> 
> > 
> > This was true in         CYGWIN_NT-10.0 2.9.0(0.318/5/3) (x86_64)
> > but is no longer true in CYGWIN_NT-10.0 2.11.1(0.329/5/3) (x86_64).
> > (Reported by Assaf Gordon.)
> > 
> > Looking at the changes that occurred in the cygwin git between
> > 2.9.0 (on 2017-08-03) and 2.11.1 (on 2018-08-31) it appears to me 
> > that the culprit is the commit
> > 41f72ab4d7c404b8ac2a5e1187d79164992e4483 (on 2018-03-09).
> > 
> > I would like to CC its author, Thomas Wolff, but he gave an invalid 
> > email address <mintty@users.noreply.github.com> in the commit.
> > 
> > This regression was found by gnulib's test suite 
> > https://git.savannah.gnu.org/gitweb/?p=gnulib.git;a=blob;f=tests/tes
> > t-wctype-h.c
> > 
> > Bruno

Thanks for the report, Bruno!


Corinna

--
Corinna Vinschen
Cygwin Maintainer
\x03B‹KCB”\x1c›Ø›\x19[H\x1c™\^[ܝ\x1cΈ\b\b\b\b\b\b\x1a\x1d\x1d\x1c\x0e‹ËØÞYÝÚ[‹˜ÛÛKÜ\x1c›Ø›\x19[\Ëš\x1d^[[\x03B‘TNˆ\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\b\x1a\x1d\x1d\x1c\x0e‹ËØÞYÝÚ[‹˜ÛÛKÙ˜\KÃB‘^[ØÝ[Y[\x18]\x1a[ÛŽˆ\b\b\b\b\b\b\b\b\x1a\x1d\x1d\x1c\x0e‹ËØÞYÝÚ[‹˜ÛÛKÙ^[ØÜËš\x1d^[[\x03B•[œÝXœØÜšX™H\x1a[™›Îˆ\b\b\b\b\b\x1a\x1d\x1d\x1c\x0e‹ËØÞYÝÚ[‹˜ÛÛKÛ[\vÈÝ[œÝXœØÜšX™K\Ú[\^[\x19CBƒB

Unsubscribing from Cygwin Mailing List (was: iswcntrl() regression)

[Brian Inglis]

On 2019-01-08 15:56, Wong, Samuel wrote:
> I have tried to unsubscribe from the Cygwin mailing list.
> I have tried simple way to unsubscribe by entering Mailing list name, Your
> email address.  But I cannot find out the mailing list name.
> I have read the unsubscribe full, but still cannot find the mailing list name.
> Please help to unsubscribe me for all mailing lists with Cygwin.
> My email address is samuel.wong@intel.com

Mailing list is Cygwin - see:

	https://cygwin.com/lists.html#subscribe-unsubscribe

	https://cygwin.com/ml/#unsubscribe-simple

also mentioned below.

-- 
Take care. Thanks, Brian Inglis, Calgary, Alberta, Canada

This email may be disturbing to some readers as it contains
too much technical detail. Reader discretion is advised.

--
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

Re: iswcntrl() regression

[Thomas Wolff]

[-- Attachment #1: Type: text/plain, Size: 2763 bytes --]

Am 06.01.2019 um 23:39 schrieb Thomas Wolff:
> Am 06.01.2019 um 21:21 schrieb Corinna Vinschen:
>> On Dec 19 21:57, Bruno Haible wrote:
>>> Hello Thomas,
>>>
>>> Oops 1: I forgot to add the reference to POSIX:
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html
>>>
>>> Oops 2:
>>> The relevant expression is iswcntrl (WEOF), not iswcntrl (EOF).
>>>
>>>>> Are there any other special requirements concerning EOF?
>>> WEOF is a special value. The difference between wint_t and wchar_t is
>>> essentially that WEOF fits into wint_t but is not guaranteed to fit 
>>> in wchar_t.
>>> (Like EOF fits into 'int' but usually does not fit in 'unsigned char'.)
>>>
>>> WEOF is special for all functions declared in <wctype.h>, see
>>>
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswalnum.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswalpha.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswblank.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswdigit.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswgraph.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswlower.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswprint.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswpunct.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswspace.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswupper.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswxdigit.html 
>>>
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswctype.html
>>>
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/towlower.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/towupper.html
>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/towctrans.html 
>>>
>>>
>>>> this could be fixed in iswcntrl_l.c as a special condition, or
>>>> injected as a special case in the categories.t table.
>>> Probably, yes.
>> Thomas, any input?  Are you going to provide patches?
> Well, yes, now I'm assuming that this is the only WEOF special case 
> and I'll look after it this week.
> Thomas
The fix should be as attached.
However, I don't seem to be able to produce a git format patch; when I 
`git format-patch -1` (after `git pull` and modifying only the affected 
file) it produces a patch file

Subject: [PATCH] Cygwin: try_to_bin: don't check recycler filename all 
the time
...
Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
---
  winsup/cygwin/syscalls.cc | 71 +++++++++++++++++++++------------------
...

which is weird.
Thomas

[-- Attachment #2: 0001-iswcntrl-WEOF.patch --]
[-- Type: text/plain, Size: 380 bytes --]

--- newlib/libc/ctype/categories.c~	2018-08-18 09:50:16.397997100 +0200
+++ newlib/libc/ctype/categories.c	2019-01-11 01:56:39.155736700 +0100
@@ -19,7 +19,7 @@ bisearch_cat(wint_t ucs, const struct _c
   int mid;
 
   if (ucs < table[0].first || ucs > table[max].first + table[max].delta)
-    return 0;
+    return -1;
   while (max >= min)
     {
       mid = (min + max) / 2;

[-- Attachment #3: Type: text/plain, Size: 219 bytes --]


--
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

Re: iswcntrl() regression

[Corinna Vinschen]

[-- Attachment #1: Type: text/plain, Size: 1568 bytes --]

On Jan 11 02:07, Thomas Wolff wrote:
> Am 06.01.2019 um 23:39 schrieb Thomas Wolff:
> > Am 06.01.2019 um 21:21 schrieb Corinna Vinschen:
> > > On Dec 19 21:57, Bruno Haible wrote:
> > > > Hello Thomas,
> > > > 
> > > > Oops 1: I forgot to add the reference to POSIX:
> > > > http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html
> > > > 
> > > > Oops 2:
> > > > The relevant expression is iswcntrl (WEOF), not iswcntrl (EOF).
> > > > [...]
> > > Thomas, any input?  Are you going to provide patches?
> > Well, yes, now I'm assuming that this is the only WEOF special case and
> > I'll look after it this week.
> > Thomas
> The fix should be as attached.
> However, I don't seem to be able to produce a git format patch; when I `git
> format-patch -1` (after `git pull` and modifying only the affected file) it
> produces a patch file
> 
> Subject: [PATCH] Cygwin: try_to_bin: don't check recycler filename all the
> time
> ...
> Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
> ---
>  winsup/cygwin/syscalls.cc | 71 +++++++++++++++++++++------------------
> ...
> 
> which is weird.

No, it's not.  Only commited patches are printed by `git format-patch',
so, after `git pull' you have to *commit* your patches, write a nice
description (with a blank line after the first line to separate the
commit message into synopsis and body).  *Then* call `git format-patch -1'.

Apart from that, please send your patch to the newlib mailing list.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: iswcntrl() regression

[Thomas Wolff]

Am 11.01.2019 um 10:16 schrieb Corinna Vinschen:
> On Jan 11 02:07, Thomas Wolff wrote:
>> Am 06.01.2019 um 23:39 schrieb Thomas Wolff:
>>> Am 06.01.2019 um 21:21 schrieb Corinna Vinschen:
>>>> On Dec 19 21:57, Bruno Haible wrote:
>>>>> Hello Thomas,
>>>>>
>>>>> Oops 1: I forgot to add the reference to POSIX:
>>>>> http://pubs.opengroup.org/onlinepubs/9699919799/functions/iswcntrl.html
>>>>>
>>>>> Oops 2:
>>>>> The relevant expression is iswcntrl (WEOF), not iswcntrl (EOF).
>>>>> [...]
>>>> Thomas, any input?  Are you going to provide patches?
>>> Well, yes, now I'm assuming that this is the only WEOF special case and
>>> I'll look after it this week.
>>> Thomas
>> The fix should be as attached.
>> However, I don't seem to be able to produce a git format patch; when I `git
>> format-patch -1` (after `git pull` and modifying only the affected file) it
>> produces a patch file
>>
>> Subject: [PATCH] Cygwin: try_to_bin: don't check recycler filename all the
>> time
>> ...
>> Signed-off-by: Corinna Vinschen <corinna@vinschen.de>
>> ---
>>   winsup/cygwin/syscalls.cc | 71 +++++++++++++++++++++------------------
>> ...
>>
>> which is weird.
> No, it's not.  Only commited patches are printed by `git format-patch',
> so, after `git pull' you have to *commit* your patches, write a nice
> description (with a blank line after the first line to separate the
> commit message into synopsis and body).  *Then* call `git format-patch -1'.
Blush; forgot about the workflow...
> Apart from that, please send your patch to the newlib mailing list.
Done.
Also sent a Unicode 11.0 update patch.
Thomas

--
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