Skip to content

Add Result #4

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

Merged
merged 12 commits into from
Dec 2, 2022
Merged

Add Result #4

merged 12 commits into from
Dec 2, 2022

Conversation

threeal
Copy link
Owner

@threeal threeal commented Dec 1, 2022

add res::Result that contains:

  • check if is contains ok or error.
  • unwrapping error.

error is saved as std::shared_ptr<std::string>, using string for simplicity and using pointer to prevent deep copy.
result contains error if is uninitialized.

@threeal threeal marked this pull request as ready for review December 2, 2022 13:01
@threeal threeal merged commit ca2f1ea into main Dec 2, 2022
@threeal threeal deleted the add-result branch December 2, 2022 13:14
@threeal threeal mentioned this pull request Mar 14, 2023
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.

1 participant