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 "signature_error_handler" method. #54

Merged
merged 1 commit into from Feb 16, 2013

Conversation

larryl
Copy link

@larryl larryl commented Sep 7, 2012

Subclasses can override signature_error_handler to do something
other than just die() with the message generated by signature_error.

Subclasses can override signature_error_handler to do something
other than just die() with the message generated by signature_error.
@schwern
Copy link
Contributor

schwern commented Feb 14, 2013

Looks like this got dropped on the floor.

Sounds like a good idea to me. +1

Perhaps it can be extended to cover other error messages we have? (Not a merge stopper)

@barefootcoder
Copy link
Contributor

Yeah, that's my fault. I was on sabbatical, and I let this stew for too long. I'll take a closer look and try to get it merged in this weekend.

@barefootcoder barefootcoder merged commit ecbc27c into evalEmpire:master Feb 16, 2013
barefootcoder added a commit that referenced this pull request Feb 17, 2013
part of cleaning up larryl's fix for #54
barefootcoder added a commit that referenced this pull request Feb 17, 2013
part of cleaning up larryl's fix for #54
barefootcoder added a commit that referenced this pull request Feb 17, 2013
part of cleaning up larryl's fix for #54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants