Re: [PATCH] void-cond: allow void in conditional expression

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

To: Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxxx>
Subject: Re: [PATCH] void-cond: allow void in conditional expression
From: Linus Torvalds <torvalds@xxxxxxxxxxxxxxxxxxxx>
Date: Wed, 2 Oct 2019 16:26:51 -0700
Cc: Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx>, Sparse Mailing-list <linux-sparse@xxxxxxxxxxxxxxx>, Randy Dunlap <rdunlap@xxxxxxxxxxxxx>
In-reply-to: <99173481-dbb9-ee7d-2b03-31aac80980e5@ramsayjones.plus.com>
References: <CAHk-=wiE8gMJhM9gDsA_ErJqvs=5T+-eRYBY7CttGhQJEZYeKQ@mail.gmail.com> <20191002212333.51017-1-luc.vanoostenryck@gmail.com> <99173481-dbb9-ee7d-2b03-31aac80980e5@ramsayjones.plus.com>

On Wed, Oct 2, 2019 at 4:11 PM Ramsay Jones <ramsay@xxxxxxxxxxxxxxxxxxxx> wrote:
>
> ... and the fact that such code exists in the kernel is, er,
> somewhat disappointing ... :(

No, I do agree. There's no reason for it in this case anyway, and it
just looks odd.

So I just committed a patch to remove the case that Randy reported.

             Linus

Follow-Ups:
- Re: [PATCH] void-cond: allow void in conditional expression
  - From: Luc Van Oostenryck

References:
- Re: conditional operator ?: usage (Linux kernel)
  - From: Linus Torvalds
- [PATCH] void-cond: allow void in conditional expression
  - From: Luc Van Oostenryck
- Re: [PATCH] void-cond: allow void in conditional expression
  - From: Ramsay Jones

Prev by Date: Re: [PATCH] void-cond: allow void in conditional expression
Next by Date: [PATCH] expand more builtins like __builtin_ffs()
Previous by thread: Re: [PATCH] void-cond: allow void in conditional expression
Next by thread: Re: [PATCH] void-cond: allow void in conditional expression
Index(es):
- Date
- Thread

[Index of Archives] [Newbies FAQ] [LKML] [IETF Annouce] [DCCP] [Netdev] [Networking] [Security] [Bugtraq] [Yosemite] [MIPS Linux] [ARM Linux] [Linux Security] [Linux RAID] [Linux SCSI] [Trinity Fuzzer Tool]