Commits · 36f7f246a827d12fa87ab86ae714754882d0abe8 · pub / google / re2

26 Feb, 2020 3 commits

Revert "Refuse to rewrite when MaxSubmatch() is too large." · 36f7f246

Paul Wankadia authored 5 years ago

This reverts commit 028429cd.

Change-Id: I50cc390e25f75211ddbae47af9a21c2c9f7b2931
Reviewed-on: https://code-review.googlesource.com/c/re2/+/52534


Reviewed-by: Paul Wankadia <junyer@google.com>

36f7f246

Fall back to NFA execution when compiling the reverse Prog failed. · eb9500c7

Paul Wankadia authored 5 years ago

Change-Id: I7336a7d7ac3310dc9a567239f898d89394803058
Reviewed-on: https://code-review.googlesource.com/c/re2/+/52532


Reviewed-by: Paul Wankadia <junyer@google.com>

eb9500c7

Refuse to rewrite when MaxSubmatch() is too large. · 028429cd

Paul Wankadia authored 5 years ago

Change-Id: I10eaa77de8f6f51e859b9da6b0bc680450ea8bc5
Reviewed-on: https://code-review.googlesource.com/c/re2/+/52530


Reviewed-by: Paul Wankadia <junyer@google.com>

028429cd

21 Feb, 2020 1 commit

Tweak the comment on RE2::QuoteMeta(). · 2c196c6f

Paul Wankadia authored 5 years ago

Change-Id: If1303cc74502f7d96e91e27af693cacd61421f12
Reviewed-on: https://code-review.googlesource.com/c/re2/+/52311


Reviewed-by: Paul Wankadia <junyer@google.com>

2c196c6f

18 Feb, 2020 1 commit

Move DeBruijnString() alongside StringGenerator. · 793b4e85

Paul Wankadia authored 5 years ago

Change-Id: Id7e2ed4be60611c0a4e56c621d9449a595c94c5e
Reviewed-on: https://code-review.googlesource.com/c/re2/+/52091


Reviewed-by: Paul Wankadia <junyer@google.com>

793b4e85

14 Feb, 2020 6 commits

Fix the check for Apple platforms. Mea culpa. · d8d0c66b

Paul Wankadia authored 5 years ago

Change-Id: I48f5974b2650b0cdb07cfef8015b6ece6b8c2111
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51981


Reviewed-by: Paul Wankadia <junyer@google.com>

d8d0c66b

Provide hooks::context iff thread_local is supported. · f2a896b2

Paul Wankadia authored 5 years ago

Change-Id: I984b825fc4831142a520db10a1f5ab9c47ced348
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51979


Reviewed-by: Paul Wankadia <junyer@google.com>

f2a896b2

Update doc/syntax.html. · 23f11526

Paul Wankadia authored 5 years ago

Fixes #235.

Change-Id: If89c3120aee012a3b40d44995abdd604cc0ad7ef
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51977


Reviewed-by: Paul Wankadia <junyer@google.com>

23f11526

MSVC also doesn't like casting lambdas with `+'. Sigh. · b0a41e52

Paul Wankadia authored 5 years ago

Change-Id: I0ac3abb775b7b69595dedc151de0d5ba79b74942
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51975


Reviewed-by: Paul Wankadia <junyer@google.com>

b0a41e52

Only the latest MSVC allows designated initalisers. · 170e3d51

Paul Wankadia authored 5 years ago

Change-Id: I34892b79ba45dbc7a7fe604da8cf6d62d87f6485
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51973


Reviewed-by: Paul Wankadia <junyer@google.com>

170e3d51

Provide instrumentation points called "hooks". · 4f5e6c26

Paul Wankadia authored 5 years ago

Change-Id: I854b2ba264d8c7239efb8a80bc440b1f5e0955f5
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51834


Reviewed-by: Randall Bosetti <rlb@google.com>
Reviewed-by: Paul Wankadia <junyer@google.com>

4f5e6c26

12 Feb, 2020 1 commit

Tidy up some GCC-only guards. · 42d10acc

Paul Wankadia authored 5 years ago

Change-Id: I3890ba72c9ec81f7f9e211d82d5cd3f87e0095b3
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51830


Reviewed-by: Paul Wankadia <junyer@google.com>

42d10acc

04 Feb, 2020 2 commits

Tidy up the SWIG guard around LazyRE2. · e17af778

Paul Wankadia authored 5 years ago

Change-Id: I03b22b235ab60723ce2f9f8686044e92b931ec27
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51413


Reviewed-by: Paul Wankadia <junyer@google.com>

e17af778

Tweak some of the comments about FilteredRE2. · beba0239

Paul Wankadia authored 5 years ago

Change-Id: Ibeff3df995487a3304470bffa8e41d8156ce5911
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51411


Reviewed-by: Paul Wankadia <junyer@google.com>

beba0239

01 Feb, 2020 1 commit

Avoid using the forward DFA in "ANCHOR_END" cases. · 05faa8db

Paul Wankadia authored 5 years ago

In UNANCHORED cases where the regexp is anchored explicitly at the end
only, we don't need the forward DFA because we already know where the
match must end! Instead, the reverse DFA can say whether there is a
match and (optionally) where it starts.

Change-Id: I52ee19626863d06b9c52dc107eb46c3ba9eabf63
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51290


Reviewed-by: Paul Wankadia <junyer@google.com>

05faa8db

30 Jan, 2020 1 commit

Make the fuzzer use FuzzedDataProvider. · 6c109e90

Paul Wankadia authored 5 years ago

Change-Id: I1af4f8d9b5b24b46d7bf74339b1fb667bcbe235f
Reviewed-on: https://code-review.googlesource.com/c/re2/+/51051


Reviewed-by: Paul Wankadia <junyer@google.com>

6c109e90

12 Jan, 2020 1 commit

Tidy up a test. · 85c01420

Paul Wankadia authored 5 years ago

Change-Id: I0d5bca1b9915fa5485d4b01796a276959788f427
Reviewed-on: https://code-review.googlesource.com/c/re2/+/50410


Reviewed-by: Paul Wankadia <junyer@google.com>

85c01420

08 Jan, 2020 1 commit

Prevent ShortVisit() from crashing fuzzers. · 5fcdf4ad

Paul Wankadia authored 5 years ago

Change-Id: I2ec82a91a806a18bc1f801a77c9efb07019babe6
Reviewed-on: https://code-review.googlesource.com/c/re2/+/50210


Reviewed-by: Paul Wankadia <junyer@google.com>

5fcdf4ad

05 Jan, 2020 1 commit

Fix a comment. · 00af5b44

Paul Wankadia authored 5 years ago

Change-Id: Ifb948abd79f37d3db9b4b119a07a6c2cec7c0a47
Reviewed-on: https://code-review.googlesource.com/c/re2/+/50010


Reviewed-by: Paul Wankadia <junyer@google.com>

00af5b44

30 Dec, 2019 1 commit

Make DFA use hints. · f71aaa0b

Paul Wankadia authored 5 years ago

Change-Id: I7b7b338a019ea19c4a99072c961dc6aa35c15915
Reviewed-on: https://code-review.googlesource.com/c/re2/+/49950


Reviewed-by: Paul Wankadia <junyer@google.com>

f71aaa0b

14 Dec, 2019 1 commit

std::is_pod<> was deprecated in C++20. · 7470f4d0

Paul Wankadia authored 5 years ago

Change-Id: I52b1b96ee49c41db201a71a58a2bb5a51d7c0e10
Reviewed-on: https://code-review.googlesource.com/c/re2/+/49590


Reviewed-by: Paul Wankadia <junyer@google.com>

7470f4d0

09 Dec, 2019 1 commit

Simplify the bytecode for the 80-10FFFF rune range. · 6a86f6b3

Paul Wankadia authored 5 years ago

Change-Id: I9185a44413375b207b5a050dda8d4c601c501eff
Reviewed-on: https://code-review.googlesource.com/c/re2/+/49270


Reviewed-by: Paul Wankadia <junyer@google.com>

6a86f6b3

25 Nov, 2019 2 commits

Fix nullptr-with-nonzero-offset bugs found by UBSan. · bb8e7775

Paul Wankadia authored 5 years ago

Change-Id: Ic94c887b3ba4211bec5f7eb096f06fe5bca53b70
Reviewed-on: https://code-review.googlesource.com/c/re2/+/48852


Reviewed-by: Paul Wankadia <junyer@google.com>

bb8e7775

Check if empty() instead of comparing size() with 0. · 4f6e1308

Paul Wankadia authored 5 years ago

Change-Id: Ie5d85102b82dcf1eee18a455eb5b8ac671b54277
Reviewed-on: https://code-review.googlesource.com/c/re2/+/48850


Reviewed-by: Paul Wankadia <junyer@google.com>

4f6e1308

11 Nov, 2019 1 commit

Fix a latent bug in Regexp::Walker<T>::Reset(). · af345599

Paul Wankadia authored 5 years ago

Change-Id: I0e2732fa6a0e2d73f20e1dc872720796cc1c22be
Reviewed-on: https://code-review.googlesource.com/c/re2/+/48110


Reviewed-by: Paul Wankadia <junyer@google.com>

af345599

29 Oct, 2019 3 commits

Tweak some printed debugging for style. · eecfdbf1

Paul Wankadia authored 5 years ago

Change-Id: Idf26536195c850db997e0ab3fc7e25735028b54e
Reviewed-on: https://code-review.googlesource.com/c/re2/+/47630


Reviewed-by: Paul Wankadia <junyer@google.com>

eecfdbf1

Address the MSVC warnings that crept in recently. · dadeb5c4

Paul Wankadia authored 5 years ago

Change-Id: I6c4be5df9b454780ba48d9c7fc40ae4e6a61114b
Reviewed-on: https://code-review.googlesource.com/c/re2/+/47594


Reviewed-by: Paul Wankadia <junyer@google.com>

dadeb5c4

Simplify parse_double_float() in util/pcre.cc. · 1911d984

Paul Wankadia authored 5 years ago

Change-Id: Ie7e7b3f12f44471d3bed3062420e6dd5e15b1ea7
Reviewed-on: https://code-review.googlesource.com/c/re2/+/47590


Reviewed-by: Paul Wankadia <junyer@google.com>

1911d984

14 Oct, 2019 1 commit

Fix the regexp_benchmark build with GNU make. · ab12219b

Paul Wankadia authored 5 years ago

Change-Id: I740a1ec5bf8130408bc39f948910664bf601d752
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46910


Reviewed-by: Paul Wankadia <junyer@google.com>

ab12219b

11 Oct, 2019 8 commits

Oops, wrap a couple of lines. · 416c764f

Paul Wankadia authored 5 years ago

Change-Id: I88d63e642062fe7e2fa3dbf23946841b18797803
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46826


Reviewed-by: Paul Wankadia <junyer@google.com>

416c764f

Read flags using the GetFlag() style. · 9311259d

Paul Wankadia authored 5 years ago

Change-Id: Ib07828b7df210f4cc8fc612be4954c19d578e2f1
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46823


Reviewed-by: Paul Wankadia <junyer@google.com>

9311259d

Make flags use the DEFINE_FLAG() style. · 7908274d

Paul Wankadia authored 5 years ago

Change-Id: I5257acf15a15d2dc6dbcc76149505c5fea5b8def
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46821


Reviewed-by: Paul Wankadia <junyer@google.com>

7908274d

Remove a condition from exhaustive1_test.cc that is no longer needed. · 84daac21

Paul Wankadia authored 5 years ago

Change-Id: I1e6fb3de3ea4ef28b3d630569eb4ae247af52cae
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46819


Reviewed-by: Paul Wankadia <junyer@google.com>

84daac21

Move util/flags.h into the testing target. · 4ca02858

Paul Wankadia authored 5 years ago

Change-Id: Ic3cf9db8ba2721fc2fc3addbf5f39386590dd93f
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46817


Reviewed-by: Paul Wankadia <junyer@google.com>

4ca02858

Don't declare testing::TempDir() in dump.cc itself. · bd0db338

Paul Wankadia authored 5 years ago

Change-Id: I029655a5509aa43494906b9a22eee25a5e8d3f3b
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46815


Reviewed-by: Paul Wankadia <junyer@google.com>

bd0db338

Split out the fake testing::MallocCounter into its own file. · 40dd3fc7

Paul Wankadia authored 5 years ago

Change-Id: I91569d8ec7280542e902af82915fb34bb8b0164c
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46812


Reviewed-by: Paul Wankadia <junyer@google.com>

40dd3fc7

Remove the fake test_tmpdir flag in favour of testing::TempDir(). · b4475075

Paul Wankadia authored 5 years ago

Change-Id: Ifa46c97eb781266e3f848b38d452b82f77aff09a
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46810


Reviewed-by: Paul Wankadia <junyer@google.com>

b4475075

10 Oct, 2019 3 commits

Remove the comment on NumCPUs(). · c7a2209b

Paul Wankadia authored 5 years ago

Change-Id: I6cdba5e3617e9c7a4b181d01b8b503c32a3ecf81
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46699


Reviewed-by: Paul Wankadia <junyer@google.com>

c7a2209b

Call .range(0) explicitly rather than just .range(). · f127d508

Paul Wankadia authored 5 years ago

Change-Id: Iaba956830b61e3a3541b4dfc21cdaf588fd6d620
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46697


Reviewed-by: Paul Wankadia <junyer@google.com>

f127d508

Move NumCPUs() into regexp_benchmark.cc. · ca01a17e

Paul Wankadia authored 5 years ago

Change-Id: Iad9429c6246a5780e2cb0f3e4f04b3b749410798
Reviewed-on: https://code-review.googlesource.com/c/re2/+/46694


Reviewed-by: Paul Wankadia <junyer@google.com>

ca01a17e