<!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>So does that mean Mark will have to give up his love for<BR>
<BR>
if (this) then = that;<BR>
<BR>
Unconsciously I do<BR>
<BR>
if (this){<BR>
   then = that;<BR>
}<BR>
<BR>
Though I can live with (except those spaces before and after this.  Do we really need that?)<BR>
<BR>
if ( this )<BR>
{<BR>
  then = that;<BR>
}<BR>
<BR>
<BR>
-----Original Message-----<BR>
From: postgis-devel-bounces@postgis.refractions.net on behalf of Paul Ramsey<BR>
Sent: Fri 12/19/2008 11:47 AM<BR>
To: PostGIS Development Discussion<BR>
Subject: Re: [postgis-devel] STYLE<BR>
<BR>
The ansi requirement unfortunately makes this worse:<BR>
<BR>
if ( this )<BR>
{<BR>
  then = that;<BR>
}<BR>
else<BR>
{<BR>
  those = them;<BR>
}<BR>
<BR>
BTW, the reason I spec'ed ansi is just because it seems to be the<BR>
majority of the existing code base. What comes out of my hands<BR>
unconciously is kr.<BR>
<BR>
if ( this ) {<BR>
  then = that;<BR>
}<BR>
<BR>
On Fri, Dec 19, 2008 at 7:41 AM, Obe, Regina <robe.dnd@cityofboston.gov> wrote:<BR>
> Alright if you like it perhaps I was a bit too harsh and that isn't the one<BR>
> 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<BR>
> programmer or some other language that relies on indentation to screw this<BR>
> up.<BR>
><BR>
> -----Original Message-----<BR>
> From: postgis-devel-bounces@postgis.refractions.net on behalf of Mark<BR>
> 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>
> ________________________________<BR>
><BR>
> The substance of this message, including any attachments, may be<BR>
> confidential, legally privileged and/or exempt from disclosure pursuant to<BR>
> Massachusetts law. It is intended solely for the addressee. If you received<BR>
> this in error, please contact the sender and delete the material from any<BR>
> computer.<BR>
><BR>
> ________________________________<BR>
><BR>
> Help make the earth a greener place. If at all possible resist printing this<BR>
> email and join us in saving paper.<BR>
><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>
><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>
<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>