Re: [Bug-apl] ⎕RE merged

2017-10-10 Thread Juergen Sauermann
Hi Elias, thanks, fixed in SVN 1013. /// Jürgen On 10/09/2017 11:46 AM, Elias Mårtenson wrote: One more issue. The last snippet in the info manual for regexp (great work, and thanks for doing it, by the way)

Re: [Bug-apl] ⎕RE merged

2017-10-10 Thread Juergen Sauermann
Hi Jay, thanks, done. Normally the doc subdir (e.g. in the  savannah SVN repsitory) contains the latest version of this file, and I sometimes (read: usuaally) forget to also commit it to the GNU web repository. /// Jürgen

Re: [Bug-apl] ⎕RE merged

2017-10-10 Thread Juergen Sauermann
Hi Elias, thanks, fixed in SVN 1013. /// Jürgen On 10/09/2017 10:11 AM, Elias Mårtenson wrote: One more bug: The call to pcre2_compile_32 should be changed from:

Re: [Bug-apl] ⎕RE merged

2017-10-10 Thread Juergen Sauermann
Hi Elias, thanks, fixed in SVN 1013. /// Jürgen On 10/09/2017 05:12 AM, Elias Mårtenson wrote: I found another bug. ↓ is used to indicate that string indexes are requested, but the error message when multiple o

Re: [Bug-apl] ⎕RE merged

2017-10-10 Thread Juergen Sauermann
Hi Elias, thanks, fixed in SVN 1013. /// Jürgen On 10/09/2017 04:41 AM, Elias Mårtenson wrote: Thank you. There are some errors when compiling on my Arch system:

Re: [Bug-apl] ⎕RE merged

2017-10-09 Thread Elias Mårtenson
One more issue. The last snippet in the info manual for regexp (great work, and thanks for doing it, by the way) looks really weird, probably because the content is too wide. Regards, Elias On 9 October 2017 at 17:02, Jay Foad wrote: > Could you please update https://www.gnu.org/software/apl/ap

Re: [Bug-apl] ⎕RE merged

2017-10-09 Thread Jay Foad
Could you please update https://www.gnu.org/software/apl/apl.html ? Or will it update automatically in due course? Thanks, Jay. On 8 October 2017 at 17:47, Juergen Sauermann wrote: > Hi, > > I have merged Elias' *⎕RE* implementation into GNU APL. > Thanks, Elias, for contributing it. See *'info

Re: [Bug-apl] ⎕RE merged

2017-10-09 Thread Elias Mårtenson
One more bug: The call to pcre2_compile_32 should be changed from: code = pcre2_compile_32(pattern_ucs, pattern.size(), PCRE2_NO_UTF_CHECK | flags, &error_code, &error_offset, 0); To: code = pcre2_compile_32(pattern_ucs, pattern.size()

Re: [Bug-apl] ⎕RE merged

2017-10-08 Thread Elias Mårtenson
I found another bug. ↓ is used to indicate that string indexes are requested, but the error message when multiple output types are requested is wrong: * "foo" ⎕RE["⊂↓"] "bar"* DOMAIN ERROR+ 'foo' ⎕RE['⊂↓']'bar' ^ ^ * )more* Multiple ⎕RE output flags: '⊂↓'. Output

Re: [Bug-apl] ⎕RE merged

2017-10-08 Thread Elias Mårtenson
I fixed the problem by adding a static_cast(len), but I found another issue: The testcases file is missing. Regards, Elias On 9 October 2017 at 10:41, Elias Mårtenson wrote: > Thank you. > > There are some errors when compiling on my Arch system: > > g++ -DHAVE_CONFIG_H -I. -I..-Wall -I sql

Re: [Bug-apl] ⎕RE merged

2017-10-08 Thread Elias Mårtenson
Thank you. There are some errors when compiling on my Arch system: g++ -DHAVE_CONFIG_H -I. -I..-Wall -I sql -Wold-style-cast -Werror -I/usr/include -I/usr/include -rdynamic -g -O2 -MT apl-Quad_RE.o -MD -MP -MF .deps/apl-Quad_RE.Tpo -c -o apl-Quad_RE.o `test -f 'Quad_RE.cc' || echo './'`Quad_

[Bug-apl] ⎕RE merged

2017-10-08 Thread Juergen Sauermann
Hi, I have merged Elias' ⎕RE implementation into GNU APL. Thanks, Elias, for contributing it. See 'info apl' for a description and src/testcases/Quad_RE.tc for examples of how to use ⎕RE. SVN 1012. Enjoy, /// Jürgen