Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add a regexp crate to the Rust distribution #42

Merged
merged 12 commits into from
Apr 22, 2014
235 changes: 235 additions & 0 deletions active/0000-regexps.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,235 @@
- Start Date: 2014-04-12
- RFC PR #: (leave this empty)
- Rust Issue #: (leave this empty)

# Summary

Add a `regexp` crate to the Rust distribution in addition to a small
`regexp_re` crate that provides a syntax extension for compiling regular
expressions during the compilation of a Rust program.

The implementation that supports this RFC is ready to receive
feedback: https://github.com/BurntSushi/regexp

Documentation for the crate can be seen here:
http://burntsushi.net/rustdoc/regexp/index.html

regex-dna benchmark (vs. Go, Python):
https://github.com/BurntSushi/regexp/tree/master/benchmark/regex-dna

Other benchmarks (vs. Go):
https://github.com/BurntSushi/regexp/tree/master/benchmark

(Perhaps the links should be removed if the RFC is accepted, since I can't
guarantee they will always exist.)

# Motivation

Regular expressions provide a succinct method of matching patterns against
search text and are frequently used. For example, many programming languages
include some kind of support for regular expressions in its standard library.

The outcome of this RFC is to include a regular expression library in the Rust
distribution.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you cc the issue in the rust repository here as well?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I added a reference to the issue. (Please let me know if that wasn't what you meant by cc.)


# Detailed design

(Note: This is describing an existing design that has been implemented. I have
no idea how much of this is appropriate for an RFC.)

The first choice that most regular expression libraries make is whether or not
to include backreferences in the supported syntax, as this heavily influences
the implementation and the performance characteristics of matching text.

In this RFC, I am proposing a library that closely models Russ Cox's RE2
(either its C++ or Go variants). This means that features like backreferences
or generalized zero-width assertions are not supported. In return, we get
`O(mn)` worst case performance (with `m` being the size of the search text and
`n` being the number of instructions in the compiled expression).

My implementation currently simulates an NFA using something resembling the
Pike VM. Future work could possibly include adding a DFA. (N.B. RE2/C++
includes both an NFA and a DFA, but RE2/Go only implements an NFA.)

The primary reason why I chose RE2 was that it seemed to be a popular choice in
issue [#3591](https://github.com/mozilla/rust/issues/3591), and its worst case
performance characteristics seemed appealing. I was also drawn to the limited
set of syntax supported by RE2 in comparison to other regexp flavors.

With that out of the way, there are other things that inform the design of a
regexp library.

## Unicode

Given the already existing support for Unicode in Rust, this is a no-brainer.
Unicode literals should be allowed in expressions and Unicode character classes
should be included (e.g., general categories and scripts).

Case folding is also important for case insensitive matching. Currently, this
is implemented by converting characters to their uppercase forms and then
comparing them. Future work includes applying at least a simple fold, since
folding one Unicode character can produce multiple characters.

Normalization is another thing to consider, but like most other regexp
libraries, the one I'm proposing here does not do any normalization. (It seems
the recommended practice is to do normalization before matching if it's
needed.)

A nice implementation strategy to support Unicode is to implement a VM that
matches characters instead of bytes. Indeed, my implementation does this.
However, the public API of a regular expression library should expose *byte
indices* corresponding to match locations (which ought to be guaranteed to be
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(The APIs in std::str expose byte indices too, so this is well supported in Rust-land.)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch! Fixed.

UTF8 codepoint boundaries by construction of the VM). My reason for this is
that byte indices result in a lower cost abstraction. If character indices are
desired, then a mapping can be maintained by the client at their discretion.

## Word boundaries, word characters and Unicode

The `\w` character class and the zero-width word boundary assertion `\b` are
defined in terms of the ASCII character set. I'm not aware of any
implementation that defines these in terms of proper Unicode character classes.
Do we want to be the first?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

\w and \d and \s all default to Unicode under Python 3. So there's a little bit of precedent.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah! I actually think D also does it. I'd say that's probably enough precedent to go with Unicode. (For word boundaries too, I think.)


## Leftmost-first

As of now, my implementation finds the leftmost-first match. This is consistent
with PCRE style regular expressions.

I've pretty much ignored POSIX, but I think it's very possible to add
leftmost-longest semantics to the existing VM. (RE2 supports this as a
parameter, but I believe still does not fully comply with POSIX with respect to
picking the correct submatches.)

## Public API

There are three main questions that can be asked when searching text:

1. Does the string match this expression?
2. If so, where?
3. Where are its submatches?

In principle, an API could provide a function to only answer (3). The answers
to (1) and (2) would immediately follow. However, keeping track of submatches
is expensive, so it is useful to implement an optimization that doesn't keep
track of them if it doesn't have to. For example, submatches do not need to be
tracked to answer questions (1) and (2).

The rabbit hole continues: answering (1) can be more efficient than answering
(2) because you don't have to keep track of *any* capture groups ((2) requires
tracking the position of the full match). More importantly, (1) enables early
exit from the VM. As soon as a match is found, the VM can quit instead of
continuing to search for greedy expressions.

Therefore, it's worth it to segregate these operations. The performance
difference can get even bigger if a DFA were implemented (which can answer (1)
and (2) quickly and even help with (3)). Moreover, most other regular
expression libraries provide separate facilities for answering these questions
separately.

Some libraries (like Python's `re` and RE2/C++) distinguish between matching an
expression against an entire string and matching an expression against part of
the string. My implementation favors simplicity: matching the entirety of a
string requires using the `^` and/or `$` anchors. In all cases, an implicit
`.*?` is added the beginning and end of each expression evaluated. (Which is
optimized out in the presence of anchors.)

Finally, most regexp libraries provide facilities for splitting and replacing
text, usually making capture group names available with some sort of `$var`
syntax. My implementation provides this too. (These are a perfect fit for
Rust's iterators.)

This basically makes up the entirety of the public API, in addition to perhaps
a `quote` function that escapes a string so that it may be used as a literal in
an expression.

## The `re!` macro

With syntax extensions, it's possible to write an `re!` macro that compiles an
expression when a Rust program is compiled. In my case, it seemed simplest to
compile it to *static* data. For example:

static re: Regexp = re!("a*");

At first this seemed difficult to accommodate, but it turned out to be
relatively easy with a type like this:

pub enum MaybeStatic<T> {
Dynamic(Vec<T>),
Static(&'static [T]),
}

Another option is for the `re!` macro to produce a non-static value, but I
found this difficult to do with zero-runtime cost. Either way, the ability to
statically declare a regexp is pretty cool I think.

Note that the syntax extension is the reason for the `regexp_re` crate. It's
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We probably should have a convention for crates and their syntax extension pairs, e.g. for a crate foo, have foo_macros or foo_synext or something. (I'd personally be ok with foo_macros, e.g. regexp_macros in this case.)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like foo_macros too. (I'll change this once there's a consensus?)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've used foo_mac but foo_macros seems fine.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the future all that will be necessary is #[phase(syntax, link)] extern crate regexp;. The compiler will automatically dynamically load the appropriate syntax extension crate, and then it will link to the target crate. Note that this is all far off, and is another reason why phase is feature gated.

Essentially, I wouldn't worry too much about the name.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK. I changed the name for now to regexp_macros. Even if it isn't necessary, I think it's probably a better name on its own than regexp_re. Happy to comply with anything though.

very small and contains the macro registration function. I'm not sure how this
fits into the Rust distribution, but my vote is to document the `re!` macro in
the `regexp` crate and hide the `regexp_re` crate from public documentation.
(Or link it to the `regexp` crate.)

## Untrusted input

Given worst case `O(mn)` time complexity, I don't think it's worth worrying
about unsafe search text.

Untrusted regular expressions are another matter. For example, it's very easy
to exhaust a system's resources with nested counted repetitions. For example,
`((a{100}){100}){100}` tries to create `100^3` instructions. My current
implementation does nothing to mitigate against this, but I think a simple hard
limit on the number of instructions allowed would work fine. (Should it be
configurable?)

## Summary

My implementation is pretty much a port of most of RE2. The syntax should be
identical or almost identical. I think matching an existing (and popular)
library has benefits, since it will make it easier for people to pick it up and
start using it. There will also be (hopefully) fewer surprises. There is also
plenty of room for performance improvement by implementing a DFA.

# Alternatives

I think the single biggest alternative is to provide a backtracking
implementation that supports backreferences and generalized zero-width
assertions. I don't think my implementation precludes this possibility. For
example, a backtracking approach could be implemented and used only when
features like backreferences are invoked in the expression. However, this gives
up the blanket guarantee of worst case `O(mn)` time. I don't think I have the
wisdom required to voice a strong opinion on whether this is a worthwhile
endeavor.

Another alternative is using a binding to an existing regexp library. I think
this was discussed in issue
[#3591](https://github.com/mozilla/rust/issues/3591) and it seems like people
favor a native Rust implementation if it's to be included in the Rust
distribution. (Does the `re!` macro require it? If so, that's a huge
advantage.)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Another small downside of binding to an existing library is that it's not necessarily as portable as rust code. Libraries written in rust are maximally portable because they'll go wherever rust goes.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah, right. Fixed.


Finally, it is always possible to persist without a regexp library.

# Unresolved questions

Firstly, I'm not entirely clear on how the `regexp_re` crate will be handled.
I gave a suggestion above, but I'm not sure if it's a good one. Is there any
precedent?

Secondly, the public API design is fairly simple and straight-forward with no
surprises. I think most of the unresolved stuff is how the backend is
implemented, which should be changeable without changing the public API (sans
adding features to the syntax).

I can't remember where I read it, but someone had mentioned defining a *trait*
that declared the API of a regexp engine. That way, anyone could write their
own backend and use the `regexp` interface. My initial thoughts are
YAGNI---since requiring different backends seems like a super specialized
case---but I'm just hazarding a guess here. (If we go this route, then we'd
probably also have to expose the regexp parser and AST and possibly the
compiler and instruction set to make writing your own backend easier. That
sounds restrictive with respect to making performance improvements in the
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We could expose it as an #[unstable] or even #[experimental] interface: i.e. subject to change, but it's possible to use if you really need it.

future.)

I personally think there's great value in keeping the standard regexp
implementation small, simple and fast. People who have more specialized needs
can always pick one of the existing C or C++ libraries.