On Thu, 07 May 2020 10:55:33 -0700
Joe Perches <[email protected]> wrote:
> > If anything, we can teach people to try to understand their fixes, to see
> > if something is really a fix or not. Blindly accepting changes like this,
> > is no different than blindly submitting patches because some tool says its
> > an issue.
>
> <shrug>
>
> Most people seem to prefer bool returns with apparent bool constants
> even though true and false are enumerator constants (int) of 1 and 0
> in the kernel.
>
> from include/linux/stddef.h:
>
> enum {
> false = 0,
> true = 1
> };
Sure, do that for new code, but we don't need these patches popping up for
current code. That is, it's a preference not a bug.
-- Steve
