Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S965038AbWH2Prg (ORCPT ); Tue, 29 Aug 2006 11:47:36 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S965037AbWH2Prf (ORCPT ); Tue, 29 Aug 2006 11:47:35 -0400 Received: from smtp.osdl.org ([65.172.181.4]:34772 "EHLO smtp.osdl.org") by vger.kernel.org with ESMTP id S964875AbWH2Pre (ORCPT ); Tue, 29 Aug 2006 11:47:34 -0400 Date: Tue, 29 Aug 2006 08:47:14 -0700 From: Andrew Morton To: Christoph Hellwig Cc: Richard Knutsson , James.Bottomley@SteelEye.com, linux-scsi@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: Conversion to generic boolean Message-Id: <20060829084714.9ae799a7.akpm@osdl.org> In-Reply-To: <20060829114502.GD4076@infradead.org> References: <44EFBEFA.2010707@student.ltu.se> <20060828093202.GC8980@infradead.org> <20060828171804.09c01846.akpm@osdl.org> <20060829114502.GD4076@infradead.org> X-Mailer: Sylpheed version 2.2.7 (GTK+ 2.8.17; x86_64-unknown-linux-gnu) Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1238 Lines: 26 On Tue, 29 Aug 2006 12:45:02 +0100 Christoph Hellwig wrote: > On Mon, Aug 28, 2006 at 05:18:04PM -0700, Andrew Morton wrote: > > At present we have >50 different definitions of TRUE and gawd knows how > > many private implementations of various flavours of bool. > > > > In that context, Richard's approach of giving the kernel a single > > implementation of bool/true/false and then converting things over to use it > > makes sense. The other approach would be to go through and nuke the lot, > > convert them to open-coded 0/1. > > > > I'm not particularly fussed either way, really. But the present situation > > is nuts. > > Let's start to kill all those utterly silly if (x == true) and if (x == false) > into if (x) and if (!x) and pospone the type decision. Adding a bool type > only makes sense if we have any kind of static typechecking that no one > ever assign an invalid type to it. Not really. bool/true/false have readability advantages over int/1/0. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/