Allocation error handling

[elfring]

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

• malloc ⇒ interactive_next
• strdup ⇒ mrsh_hashtable_set

[emersion]

Yeah, this is a TODO. I'd merge patches to do so. Adding this to the core mrsh library has higher priority than adding it to executables.

[elfring]

How do you think about to improve static source code analysis also for this software?

[emersion]

What do you mean?

I run Clang's static analyzer from time to time. I'd prefer not to run it for each pull request because it's slow and has false positives. Do you have better ideas?

Related: I also run sanitizers as part of CI: Arch builds run tests with ASan and UBSan enabled.

[elfring]

• 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?

[emersion]

No, sorry, I'd prefer to stick with the current style. Readability and KISS first.

[ddevault]

Could run static analysis on each commit rather than each PR. Would require some builds.sr.ht changes.