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

Completion of error handling #10

Open
elfring opened this issue Aug 7, 2014 · 4 comments
Open

Completion of error handling #10

elfring opened this issue Aug 7, 2014 · 4 comments
Labels
enhancement potential new feature

Comments

@elfring
Copy link

elfring commented Aug 7, 2014

I have looked at a few source files for your current software. I have noticed that some checks for return codes are missing.

Would you like to add more error handling for return values from functions like the following?

@dominicgs
Copy link
Contributor

We definitely need to do this sort of thing. The code was originally a proof of concept and we were aware that we weren't bothering to check return values, etc. Now that it's getting a little more attention I think it's about time we cleaned things up.

@elfring
Copy link
Author

elfring commented Aug 7, 2014

Are you interested to apply aspect-oriented software development?
How do you think about to encapsulate error detection and corresponding exception handling as a reusable aspect in your software?

@dominicgs
Copy link
Contributor

I don't know enough about aspect oriented programming to answer your question, but I'll try to read about it and see how it applies to USBProxy.

@elfring
Copy link
Author

elfring commented Aug 12, 2014

Do you find information sources like the following useful?

BenGardiner pushed a commit to BenGardiner/USBProxy that referenced this issue Nov 29, 2015
new buttonsme script for reading arduino serial stream and injecting evdev events
@straithe straithe added the enhancement potential new feature label Nov 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement potential new feature
Projects
None yet
Development

No branches or pull requests

3 participants