11 Feb
2025
11 Feb
'25
3:58 p.m.
On Tue, Feb 11, 2025 at 10:50:33AM -0500, Tamir Duberstein wrote:
On Tue, Feb 11, 2025 at 10:42 AM Andy Shevchenko andriy.shevchenko@linux.intel.com wrote:
On Tue, Feb 11, 2025 at 10:13:37AM -0500, Tamir Duberstein wrote:
The test already prints the same information on failure; remove redundant pr_debug() logs.
...
pr_debug("\"%s\", \"%s\" ->\n", str, fmt); \What *if* the n_args == 0 here?
Then there's no assertion in this block, so the test cannot possibly fail here.
Correct, but I'm talking about this in a scope of the removed debug print. I.o.w. how would we even know that this was the case?
(I'm not objecting removal, what I want from you is to have a descriptive and explanatory commit message that's answers to "why is this needed?" and "why is it safe to do?")
--
With Best Regards,
Andy Shevchenko