On Tue, 22 Feb 2022 08:59:46 -0800 Beau Belgrave <beaub@xxxxxxxxxxxxxxxxxxx> wrote: > > Thoughts? > > Many times we have found that we have to do work that should only be > performed if the event is enabled. As long as there is a still clear > ways to check without the macro, this would be fine with me. > > IE: strlen(message) > > We should not walk strings unless the event is enabled. Right. The macro would only be a helper macro. I would even add it as a separate patch at the end and not part of the main patch. -- Steve