On 4/17/21 3:56 AM, Philip Leishman wrote:
In the comments Ted mentions the problem with RG_MODULE_STRING. This
seems to be an ordering problem.

Ah! I knew it was something like that. I just didn't have time to dig into it. Good find there.

2. remove the generic definition in Debug.h

I think this is a great idea since it will indeed force us to get this right everywhere immediately instead of stumbling on it later as we do now.

Also the debug
macros should then not be used in header files - Event.h does this

That's always going to be a problem. Depending on the situation, either move to .cpp or remove the RG_DEBUG (especially for things that we actual want to compile as inline).

Ted.


_______________________________________________
Rosegarden-devel mailing list
Rosegarden-devel@lists.sourceforge.net - use the link below to unsubscribe
https://lists.sourceforge.net/lists/listinfo/rosegarden-devel

Reply via email to