Skip to content

remove is_match free function #165

Closed
@BurntSushi

Description

@BurntSushi

I think it was unwise to ever add this and I didn't give much thought to it. I saw it as a convenience for just quickly matching a regex against some text, but:

  1. It's more explicit to just create the Regex value and us that to match.
  2. It encourages compiling the regex every time it's used to search text, which is an anti-pattern because compilation is much slower relative to execution.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions