Re: [PATCH 01/16] selftests/mm: remove argc and argv unused parameters

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, 9 Jan 2025 09:50:45 -0800 Kees Cook <kees@xxxxxxxxxx> wrote:

> On Thu, Jan 09, 2025 at 10:48:52PM +0500, Muhammad Usama Anjum wrote:
> > For the all other case, why should we keep argv/argc and mark them unused
> > as well when they aren't being used?
> 
> I'm fine either way, but my personal code style instinct is to keep the
> "standard" main declaration with argc/argv present. But it's mostly
> aesthetic.
> 
> And if you think use of kselftest.h isn't universal, then perhaps we can
> avoid the macro, but it does seem nicer and more "normal" feeling for
> the rest of kernel development.
> 

Agreed.  __attribute__((unused)) is a bit of a mouthful and isn't what
the kernel developer's eye expects to see.  




[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Bugtraq]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]

  Powered by Linux