Relay-Version: version B 2.10 5/3/83; site utzoo.UUCP
Posting-Version: version B 2.10.2 9/18/84; site wateng.UUCP
Path: utzoo!watmath!wateng!ksbszabo
From: ksbszabo@wateng.UUCP (Kevin Szabo)
Newsgroups: net.lang.c
Subject: Re: parens around sizeof arg
Message-ID: <1993@wateng.UUCP>
Date: Sat, 9-Feb-85 22:58:11 EST
Article-I.D.: wateng.1993
Posted: Sat Feb  9 22:58:11 1985
Date-Received: Sat, 16-Feb-85 05:33:45 EST
References: <8048@brl-tgr.ARPA>
Reply-To: ksbszabo@wateng.UUCP (Kevin Szabo)
Organization: VLSI Group, U of Waterloo
Lines: 23
Summary: reply to gross construct shown by (who else) cottrell

In article <8048@brl-tgr.ARPA> cottrell@nbs-vms.ARPA writes:
>	if (exp)			if (exp) {
>		return exp;			exp;
>						return;
>					}
>
>The example on the left avoids the braces while semi-documenting
>the fact that. I know, you are shocked! I'm just a bad guy.

Sir, you are a sinner. I hope I never have to look at your code. ( 1/2 :-) )
The code on the left tells me that you are trying to pass back
a value to the calling routine, and I would be confused as to why
the calling routine chose to ignore it. If you insert a comment
to document the fact you are doing something odd, why not just do
it right the first time and avoid the comment? Of course you may
not put in any comments, but I amazed that you can figure out what
your code does three months after you wrote it. 

You really should be coding in B, or assembler.

					Kevin
-- 
Kevin Szabo  watmath!wateng!ksbszabo (U of Waterloo VLSI Group, Waterloo Ont.)