search

osmwp / MeMoPlayer

VRML/JS engine for Java enabled phones
http://github.com/osmwp/MeMoPlayer/wiki
4 stars 3 forks source link

Completion of error handling #7

Open elfring opened 12 years ago

elfring commented 12 years ago

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 commented 12 years ago

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 commented 12 years ago

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?