Skip to content
This repository has been archived by the owner on Apr 24, 2019. It is now read-only.

Completion of error handling #7

Open
elfring opened this issue Mar 14, 2012 · 2 comments
Open

Completion of error handling #7

elfring opened this issue Mar 14, 2012 · 2 comments

Comments

@elfring
Copy link

elfring commented Mar 14, 2012

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?

@yalp
Copy link

yalp commented Mar 14, 2012

Are you running a static analysis tool on the code ?
Would be great if you could contribute a PR for all these errors if you detected more (all?) of them.

@elfring
Copy link
Author

elfring commented Mar 14, 2012

I did not run a tool for automatic static code analysis on your source files. I have seen more update candidates (memory management and file handling functions) during my short manual source code inspection.

Would you like to reduce the efforts for error code checking by an exception class hierarchy?

How do you think about to apply aspect-oriented software development?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants