loot / libloot / 16578711698

93%

master: 94%

Build Type

push

github

Committed by Ortham

Commit Message

Switch to the regress crate for regex

It uses about 56 MB less memory when LOOT launches into my Skyrim SE
large test load order with the latest masterlist and prelude, and
explicitly targets ECMAScript-like syntax like C++ std::regex does.

Although the doc update now says that modifiers and some Unicode
property escapes are unsupported, that's no more restrictive than when
libloot was using C++ std::regex, which also doesn't support modifiers
and doesn't support *any* Unicode property escapes.

What is a new restriction is the lack of support for ASCII character
classes (a.k.a. POSIX-based character classes in C++), which were
supported by fancy-regex and C++ std::regex. A search of the masterlists
doesn't find any use of them, and it's very unlikely that any user
metadata would either.

Run Details

35 of 37 new or added lines in 3 files covered. (94.59%)

11 existing lines in 1 file now uncovered.

10655 of 11470 relevant lines covered (92.89%)

217.08 hits per line