On 4/18/21 3:48 PM, Philip Leishman wrote:
I still think qCDebug is a good way to go. I just don't know how to do
it without changing many hundreds of lines of code in almost all files.
Before we embark on this project, I would like to see a proof of
concept that shows that we can correctly preserve the behavior of the
current RG_* macros with whatever the replacement is. Here are the
requirements based on the existing macros:
RG_DEBUG happens only in a debug build. It can be turned off with
RG_NO_DEBUG_PRINT.
RG_WARNING happens in both builds and is always on by default.
RG_INFO happens only in a debug build and is always on by default.
(This one is barely used at all and can probably be dropped.)
The proof of concept could easily be run from main(). It already
issues RG_INFO and RG_DEBUG messages. It would just need a test
RG_WARNING added:
RG_WARNING << "main(): testing RG_WARNING";
Then replace everything in main.cpp with the new approach and see if
it can meet my requirements above.
Once I see that, I will be more likely to be on board. Otherwise I
would rather that we leave things as-is. The current RG_* macros work
fine for me and I see no benefit to changing them. All I see is time
wasted fiddling with debug logging that already works.
Ted.
_______________________________________________
Rosegarden-devel mailing list
Rosegarden-devel@lists.sourceforge.net - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel
