Remove globals and refactor to complete implementation of logging.

[wargio]

Your checklist for this pull request

• I've read the guidelines for contributing to this repository
• I made sure to follow the project's coding style
• I've documented or updated the documentation of every function and struct this PR changes. If not so I've explained why.
• I've added tests that prove my fix is effective or that my feature works (if possible)
• I've updated the rizin book with the relevant information (if needed)

Detailed description

• Removes almost all the globals from the logging api
• Logging C API are now thread-safe.
• Implements colored logs and more.
• RZ_LOG_SILLY has been removed.
• Should be a bit faster since fprintf has been converted to fputs.
• Renamed log.traplevel to log.abortlevel and hide it in release builds.

[XVilka]

Nice finding. Apart from Doxygen - LGTM

[Rot127]

While you are at it. Could you add a simple getter for the current log level? Plugins can check it, when they print log messages.

Or, even better, add a logging function which takes the message as a string (not with vargs). Because vargs cannot be used from a few languages.

[Rot127]

Actually, if you do spent the time to add the logging API for plugins, it might be a good idea add an additional argument, where plugins can pass their name.
So the log messages can be distinguished from main Rizin.

[INFO] [Plugin] ...

[wargio]

Actually, if you do spent the time to add the logging API for plugins, it might be a good idea add an additional argument, where plugins can pass their name. So the log messages can be distinguished from main Rizin.

[INFO] [Plugin] ...

tbh, there is an option which tells you the function, file and line where that occurs.