Re: [PATCH] regex: fix assertion in re_node_set_insert

2021-06-22 Thread Paul Eggert
Thanks, I installed that into Gnulib in your name.

Re: inflexible non-recursive support

2021-06-22 Thread Paul Eggert
On 6/22/21 9:33 AM, Werner LEMBERG wrote: I suggest a patch as below (which needs further polishing to properly canonicalize the directory path, I guess). Thanks for the suggestion. Could you spell out a little more what sort of polishing would be needed there? (I'm not a regular user of this

inflexible non-recursive support

2021-06-22 Thread Werner LEMBERG
[c7b1e060d17023065c776757da406d728310cc38] Dear gnulib maintainers, Currently, the script `prefix-gnulib-mk` replaces variables like `libbison_a_SOURCES` with `lib_libbison_a_SOURCES`. However, this works only if the directory is indeed `lib/', which is not the general case. For example, in

Re: [PATCH] regex: fix match with possessive quantifier

2021-06-22 Thread Egor Ignatov
> regex_internal.c:1317:7: runtime error: execution reached an unreachable > program point > FAIL test-regex (exit status: 1) This problem occurs in this test: '{ "()\\1*\\1*", "", REG_EXTENDED, 2, { { 0, 0 }, { 0, 0 } } }' because proceed_next_node tries to insert an existing element into the se

[PATCH] regex: fix assertion in re_node_set_insert

2021-06-22 Thread Egor Ignatov
* lib/regexec.c (proceed_next_node): Add duplicate insertion check for eps_via_nodes set. Signed-off-by: Egor Ignatov --- lib/regexec.c | 10 ++ 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/lib/regexec.c b/lib/regexec.c index 23b984a21..c05b92783 100644 --- a/lib/regexec