Skip to content
This repository was archived by the owner on Nov 9, 2019. It is now read-only.

Reuse errno_from_ioerror to simplify error handling #46

Merged
kubkon merged 1 commit intoCraneStation:masterfrom
marmistrz:error-refactor
Aug 8, 2019
Merged

Reuse errno_from_ioerror to simplify error handling #46
kubkon merged 1 commit intoCraneStation:masterfrom
marmistrz:error-refactor

Conversation

@marmistrz
Copy link
Copy Markdown
Member

This PR bases on #42.

@kubkon
Copy link
Copy Markdown
Member

kubkon commented Aug 5, 2019

LGTM! If you don't mind, I'd like to have #41 squashed and rebased onto master first, before we proceed with this one. Is that OK with you? :-)

@marmistrz
Copy link
Copy Markdown
Member Author

marmistrz commented Aug 5, 2019

ok :)

@kubkon
Copy link
Copy Markdown
Member

kubkon commented Aug 8, 2019

@marmistrz OK, so #41 is now merged. Could you rebase and apply your refactoring to anything that's either new or missed? :-)

@marmistrz
Copy link
Copy Markdown
Member Author

marmistrz commented Aug 8, 2019

I updated the change

@kubkon
Copy link
Copy Markdown
Member

kubkon commented Aug 8, 2019

Great, thanks!

@kubkon kubkon merged commit 92c2b56 into CraneStation:master Aug 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants