On 10/21/2022 12:17 PM, Andrew Morton wrote:
On Fri, 21 Oct 2022 11:26:22 +0800 "Aiqun(Maria) Yu" <quic_aiquny@xxxxxxxxxxx> wrote:
Honestly, the cure is worse than the disease. I'd rather not have a
line that's this long.
I don't like this long either. While the current Deadcode warning is
also annoying for our clang tidy check.
will the CONFIG check can be acceptable?
like:
#ifdef CONFIG_DEBUG_VM
or
if (IS_ENABLED(CONFIG_DEBUG_VM))
Any good ideas?
Put a newline in there?
Although the shortly following lines are nearly as long...
Thx for the suggestion. new patchset uploaded.
--
Thx and BRs,
Aiqun(Maria) Yu
