[Lldb-commits] [PATCH] D27459: Straw-man proposal for new logging syntax

[Greg Clayton via Phabricator via lldb-commits]

clayborg added a comment.

I am fine with us changing logging, just want to make sure it works well. Ideally it would include:

- almost zero cost when logging disabled. No arguments should be passed to any functions and then determine that logging is disabled, it should check if logging is enabled and not do any of the work if it isn't.
- no state in streams so adding logging before other logging can't affect future output
- the ability to some form if format strings (printf for formatv style strings) so not everything has to be done with streams and control codes that are sent into the stream to change from decimal to hex  etc
- I would rather avoid macros if possible, though we might have to use them to get the performance


https://reviews.llvm.org/D27459