On Tue Apr 11 11:37:34 2023 +0000, Nikolay Sivov wrote:
Since you're changing this, let's improve formatting a bit, by getting rid of This (replace with 'view', 'shellview', 'folderview', etc), and using iface pointer in trace messages, with other arguments simply being comma-separated. So no braces or "->".
I never understood the "new format" for traces, using `iface`. Is there a reason for it? It makes looking through logs harder for no reason at all. I much prefer tracing `This` (the object itself) so that I can search for the same pointer value to find use of that object in logs. Old-style traces are much easier for me to debug.
You already know which iface it's using purely from the function name in the log, anyway…