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.

Sign up for GitHub

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

Jump to bottom

Adds PartialEq impl for Regex. #378

Closed

remexre wants to merge 1 commit into rust-lang:master from remexre:partialeq

Contributor

remexre commented Jun 1, 2017

This simply compares the two regexes' strings, as the same string should always produce the same regex.


          Adds PartialEq impl for Regex.

6ebaf4f

Member

BurntSushi commented Jun 1, 2017

Thanks, but this was explicitly dropped in regex 0.2 because regex equivalence is not determined by their concrete syntax. See #178, #313 and #364

BurntSushi closed this

remexre deleted the partialeq branch

June 1, 2017 15:35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet