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.

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

Rewrite CompareMessage to check the whole string #611

Merged
merged 1 commit into from
May 23, 2019
Merged

Rewrite CompareMessage to check the whole string #611

merged 1 commit into from
May 23, 2019

Conversation

phil-opp
Copy link
Owner

Instead of checking just a single string component like before.

Fixes the issues reported in #591 (comment). Alternative to #610.

cc @xj42

@phil-opp
Rewrite CompareMessage to check the whole string
5580c2b 
Not just a single string component like before.
This was referenced May 22, 2019
Closed
Closed
@phil-opp phil-opp merged commit 99f8d2c into master May 23, 2019
@bors bors bot deleted the panic-handler-test branch May 23, 2019 10:40
phil-opp added a commit that referenced this pull request May 23, 2019
@phil-opp
Update code for PR #611
c8d1bbe 
PR at #611
@phil-opp
Copy link
Owner Author

phil-opp commented May 23, 2019 •
edited
Loading

Code update in c8d1bbe.

@phil-opp phil-opp added the relnotes "Release notes" – Notable changes that are rendered on the blog. label Jun 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
relnotes "Release notes" – Notable changes that are rendered on the blog.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@phil-opp