Think about defining `LOG_DECLARE_ISSUE` macros in Logging.hpp

[eflumerf]

Since #include "logging/Logging.hpp" must be visible to translation units before ERS_DECLARE_ISSUE is called, it may be a good idea to define macros in Logging.hpp of the following form:

#define LOG_DECLARE_ISSUE(...) ERS_DECLARE_ISSUE(va_args)
#define LOG_DECLARE_ISSUE_BASE(...) ERS_DECLARE_ISSUE_BASE(va_args)
...

The idea would be that everything then uses a "logging-style" interface, and reduce confusion about what needs to be included to make ers::Issue classes.

[ron003]

I agree. I like the idea!

[eflumerf]

Assigning to v5.x.x as this will have a large impact to actually use.