twin/ripgrep
1
0
Fork 0
mirror of https://github.com/BurntSushi/ripgrep.git synced 2025-05-19 01:30:21 -07:00
Code Issues Packages Projects Releases Wiki Activity

regex: update regression test

Browse Source 
Sadly, PCRE2 has different behavior (but doesn't panic). We should look
into that, but for now, this is good enough.

Also, update the CHANGELOG.

Ref #1891
This commit is contained in:
Andrew Gallant 2021-06-12 16:22:30 -04:00
parent bc76a30c23
commit 7ce66f73cf
No known key found for this signature in database
GPG Key ID: B2E3A4923F8B0D44
2 changed files with 14 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
CHANGELOG.md
View File

@ -1,7 +1,12 @@
TBD 13.0.1
=== ======
Unreleased changes. Release notes have not yet been written. Unreleased changes. Release notes have not yet been written.
Bug fixes:
* [BUG #1891](https://github.com/BurntSushi/ripgrep/issues/1891):
Fix bug when using `-w` with a regex that can match the empty string.
13.0.0 (2021-06-12) 13.0.0 (2021-06-12)
=================== ===================

7
tests/regression.rs
View File

@ -1032,6 +1032,13 @@ rgtest!(r1878, |dir: Dir, _: TestCommand| {
// See: https://github.com/BurntSushi/ripgrep/issues/1891 // See: https://github.com/BurntSushi/ripgrep/issues/1891
rgtest!(r1891, |dir: Dir, mut cmd: TestCommand| { rgtest!(r1891, |dir: Dir, mut cmd: TestCommand| {
// TODO: Sadly, PCRE2 has different behavior here. Not clear why. We should
// look into this and see if there's a fix needed at the regex engine
// level.
if dir.is_pcre2() {
return;
}
dir.create("test", "\n##\n"); dir.create("test", "\n##\n");
// N.B. We use -o here to force the issue to occur, which seems to only // N.B. We use -o here to force the issue to occur, which seems to only
// happen when each match needs to be detected. // happen when each match needs to be detected.