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

Improve the error model #101

Merged
merged 4 commits into from
Jun 8, 2014
Merged

Improve the error model #101

merged 4 commits into from
Jun 8, 2014

Conversation

jacquesg
Copy link
Owner

It's not always possible (or preferred) to deduce exactly what went wrong with a plain old string error. libgit2 can provide the error class and error code in addition to an error message.

  • Implement a first class error object that captures the error code, class and message.
  • Overload the "" operator to print the error message so existing code just works
  • Expose the error code and error class constants to the user

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.4%) when pulling 3e4f9fe on jacquesg/error-model into b1cbbf8 on master.

@jacquesg jacquesg changed the title WIP: Improve the error model Improve the error model Jun 8, 2014
@jacquesg jacquesg merged commit 3e4f9fe into master Jun 8, 2014
@jacquesg jacquesg deleted the jacquesg/error-model branch August 5, 2014 18:22
@jacquesg jacquesg restored the jacquesg/error-model branch August 5, 2014 18:23
@jacquesg jacquesg deleted the jacquesg/error-model branch August 5, 2014 18:23
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

2 participants