Re: Lookaround assertions in regexps

2021-01-22 Thread Paul Eggert
On 1/22/21 1:00 PM, Brian Leung wrote: Tomohiro Matsui wrote a patch to support lookaround assertions in regexps some time ago (http://emacs.1067599.n8.nabble.com/Patch-for-lookaround-assertion-in-regexp-td121057.html#a23863009), but it never got merged. Is there still interest in something l

Lookaround assertions in regexps

2021-01-22 Thread Brian Leung
Tomohiro Matsui wrote a patch to support lookaround assertions in regexps some time ago (http://emacs.1067599.n8.nabble.com/Patch-for-lookaround-assertion-in-regexp-td121057.html#a23863009), but it never got merged. Is there still interest in something like this? If so I can try updating the p

Re: ptsname_r on Cygwin

2021-01-22 Thread Ken Brown
On 1/21/2021 5:45 PM, Ken Brown wrote: On 1/21/2021 3:26 PM, Bruno Haible wrote: Ken Brown wrote: Thanks for the report.  I've submitted a patch that fixes this:     https://cygwin.com/pipermail/cygwin-patches/2021q1/010995.html Thanks. EBADF is the more "proper" errno value here, for fd < 0

Re: [PATCH 4/7] stdlib: putenv() needs a cast on OS/2 kLIBC

2021-01-22 Thread KO Myung-Hun
Hi/2, Bruno! Bruno Haible wrote: > Hi KO, > >> On OS/2 kLIBC, the first parameter of putenv () is `const char *string' >> not `char *string'. > > Can't you fix that in OS/2 kLIBC? Then a workaround in Gnulib would not be > needed. > I have no such a right. Instead I'll suggest that to maintain

Re: [PATCH 1/7] spawn: Use special invocation for on OS/2 kLIBC

2021-01-22 Thread KO Myung-Hun
Hi, Bruno. Bruno Haible wrote: > Hi KO, > > Thanks, I applied patches 1, 2, 3, 5, 6, 7, slightly modified. Some of the > things that you put into the git commit message actually better belong in > the code, as a comment. > Ok, I'll do so later. Thanks for your effort! -- KO Myung-Hun Using

Re: [PATCH 5/7] zerosize-ptr: Fix compilation on OS/2 kLIBC

2021-01-22 Thread KO Myung-Hun
Hi/2. Bruno Haible wrote: > Applied with modifications: >> -#if HAVE_SYS_MMAN_H && HAVE_MPROTECT >> +#if HAVE_SYS_MMAN_H && HAVE_MPROTECT && (!defined __KLIBC__ || HAVE_MMAP) > > There's a comment two lines above, that explains why HAVE_MMAP is not > the right test here. And since you write "OS/2