toxcore
toxcore copied to clipboard
irungentoo
Check return codes everywhere
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?
- fclose ⇒ terminate_logger
- ftell ⇒ load_file
- pthread_cond_signal ⇒ toxav_kill_transmission
- pthread_mutex_init ⇒ create_crypto_connection
I suggest to avoid ignorance of return values a bit more. Would you like to detect every error situation as early as possible?
closed for not having any activity in a year... reopened, sorry it got caught in crossfire
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?
@elfring is there an AOP framework for C? Or does AspectC++ work for C code?
is there an AOP framework for C?
How do you think about to look into the archives for corresponding approaches once more?
Or does AspectC++ work for C code?
- Would you like to achieve any further software improvements around answers for a question like "What about pure C code (no C++)?" for this development tool?
- Do you find the available information promising?
Ok, so https://github.com/TokTok/toxcore/pull/143 would solve the "However, the AspectC++ parser is a C++ parser and there are a few incompatibilities" issue. It generating C++ code means our code becomes effectively C++, and can no longer be compiled by simple C compilers. Right now, it seems that toxcore doesn't support platforms for which no C++ compiler exists, but the fact that it's written in C to begin with indicates an intention to support them.
Do you know whether the "pure C aspectc" that was planned for 2006 actually happened?
The manual that comes with the most recent version (2016, 10 years later) says:
5.2.7 Support for Plain C Code
Currently ac++ generates C++ code, which cannot be compiled by a C compiler.
As for many hardware platforms in the embedded domain no C++ compiler is
available we are actively looking for a solution.
So I guess not. And I don't know what "actively looking for a solution" means anymore.
And I don't know what "actively looking for a solution" means anymore.
translation: We posted about it on the internet once, and no one got back to us yet...