<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 3.2//EN">
<HTML>
<HEAD>
<META HTTP-EQUIV="Content-Type" CONTENT="text/html; charset=iso-8859-1">
<META NAME="Generator" CONTENT="MS Exchange Server version 6.5.7653.38">
<TITLE>RE: [postgis-devel] STYLE</TITLE>
</HEAD>
<BODY>
<!-- Converted from text/plain format -->
<P><FONT SIZE=2>Alright if you like it perhaps I was a bit too harsh and that isn't the one that bothers me the most. How about this concession.<BR>
<BR>
//OKAY<BR>
if (!binary) free(v);<BR>
<BR>
//NOT OKAY<BR>
if (!binary)<BR>
free(v);<BR>
<BR>
//NOT OKAY<BR>
if (!binary) free(v); printf(...);<BR>
<BR>
<BR>
The issue I have with the NOT OKAY is that its too easy for a python programmer or some other language that relies on indentation to screw this up.<BR>
<BR>
-----Original Message-----<BR>
From: postgis-devel-bounces@postgis.refractions.net on behalf of Mark Cave-Ayland<BR>
Sent: Fri 12/19/2008 10:34 AM<BR>
To: PostGIS Development Discussion<BR>
Subject: Re: [postgis-devel] STYLE<BR>
<BR>
Obe, Regina wrote:<BR>
<BR>
> This I mentioned to Mark offlist, but since its sort of a STYLE issue I<BR>
> thought I would restate it here.<BR>
><BR>
> I know C programmers are in love with doing this short-cut.<BR>
><BR>
> if (!binary) free(v);<BR>
><BR>
> I hate it. It drives me crazy because its too prone to error when<BR>
> someone comes<BR>
> by and tries to stuff another statement in there and forgets to put the<BR>
> bracket in.<BR>
<BR>
Aaargh - I really like this and use it a lot :(<BR>
<BR>
The convention is fairly easy to read: if there are no {}s then only the<BR>
next statement is considered part of the if... or else... clause. Does<BR>
that help at all?<BR>
<BR>
<BR>
ATB,<BR>
<BR>
Mark.<BR>
<BR>
--<BR>
Mark Cave-Ayland<BR>
Sirius Corporation - The Open Source Experts<BR>
<A HREF="http://www.siriusit.co.uk">http://www.siriusit.co.uk</A><BR>
T: +44 870 608 0063<BR>
_______________________________________________<BR>
postgis-devel mailing list<BR>
postgis-devel@postgis.refractions.net<BR>
<A HREF="http://postgis.refractions.net/mailman/listinfo/postgis-devel">http://postgis.refractions.net/mailman/listinfo/postgis-devel</A><BR>
<BR>
</FONT>
</P>
</BODY>
</HTML>
<HTML><BODY><P><hr size=1></P>
<P><STRONG>
The substance of this message, including any attachments, may be confidential, legally privileged and/or exempt from disclosure pursuant to Massachusetts law. It is intended solely for the addressee. If you received this in error, please contact the sender and delete the material from any computer.
</STRONG></P></BODY></HTML>
<P><hr size=1></P>
<P><STRONG><font size="2" color="339900"> Help make the earth a greener place. If at all possible resist printing this email and join us in saving paper. </p> <p> </font></STRONG></P>