From owner-freebsd-current Thu Jan 28 11:56:54 1999 Return-Path: Received: (from majordom@localhost) by hub.freebsd.org (8.8.8/8.8.8) id LAA03129 for freebsd-current-outgoing; Thu, 28 Jan 1999 11:56:54 -0800 (PST) (envelope-from owner-freebsd-current@FreeBSD.ORG) Received: from whistle.com (s205m131.whistle.com [207.76.205.131]) by hub.freebsd.org (8.8.8/8.8.8) with ESMTP id LAA03124 for ; Thu, 28 Jan 1999 11:56:50 -0800 (PST) (envelope-from archie@whistle.com) Received: (from smap@localhost) by whistle.com (8.7.5/8.6.12) id LAA08399; Thu, 28 Jan 1999 11:56:36 -0800 (PST) Received: from bubba.whistle.com( 207.76.205.7) by whistle.com via smap (V2.0) id xma008388; Thu, 28 Jan 99 11:56:13 -0800 Received: (from archie@localhost) by bubba.whistle.com (8.8.7/8.6.12) id LAA22422; Thu, 28 Jan 1999 11:56:12 -0800 (PST) From: Archie Cobbs Message-Id: <199901281956.LAA22422@bubba.whistle.com> Subject: Re: btokup().. patch to STYLE(9) (fwd) In-Reply-To: from Julian Elischer at "Jan 28, 99 03:38:31 am" To: julian@whistle.com (Julian Elischer) Date: Thu, 28 Jan 1999 11:56:12 -0800 (PST) Cc: current@FreeBSD.ORG X-Mailer: ELM [version 2.4ME+ PL38 (25)] MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Sender: owner-freebsd-current@FreeBSD.ORG Precedence: bulk X-Loop: FreeBSD.ORG Julian Elischer writes: > Some people when confronted by people wanting to have extra braces > say "change style(9)". > > Well, here is my change.. You can count my vote. I would also add a paragraph like this: If possible code should complile cleanly with gcc's -Wall flag. Note however that this does not imply that it's OK to eliminate warnings simply by covering them up with typecasts, etc., as that actually does more harm than good. I hope that wording is sufficiently unoffensive to the -Wall haters. -Archie ___________________________________________________________________________ Archie Cobbs * Whistle Communications, Inc. * http://www.whistle.com To Unsubscribe: send mail to majordomo@FreeBSD.org with "unsubscribe freebsd-current" in the body of the message