[postgis-users] 1.3.3 Pre-Release Notice

[Paragon Corporation]

 I thought about that too.   If I were making minor changes (which I had
hoped this would be), I wouldn't have bothered.  I only reindented the
sections I was working on because it was getting in my way of understanding
what was going on.  Most of it was indented fine but just some critical long
case and nested if sections.

Thanks,
Regina

-----Original Message-----
From: postgis-users-bounces at postgis.refractions.net
[mailto:postgis-users-bounces at postgis.refractions.net] On Behalf Of David
Potts
Sent: Wednesday, April 09, 2008 2:57 AM
To: PostGIS Users Discussion
Subject: RE: [postgis-users] 1.3.3 Pre-Release Notice

 The other issue I noticed with the file, and could just be my editor
> (which
> drove me a bit crazy) is that in some locations, the indentation is 
> off so I reindented some sections.  Also less of a deal there seems to 
> be a mix of syntax style -e.g
>
> if () {
>
> }
>
> Vs.
> if (..)
> {
>
> }
>
> What is the prefered style?
>
> Thanks,
> Regina
>
Reindenting the code may be nice and may help people to read the code, but
its not a wise idea.

If a new bug is introduced in to the code, its easier to find the areas that
changed between releases, if you just look at the changes made. 
Reindenting the code great increases the number of changes lines, thus
making it harder to spot any problems.

D.


--
Any views expressed in this message are those of the individual sender,
except where the sender specifically states them to be the views of the
Pinan Software


_______________________________________________
postgis-users mailing list
postgis-users at postgis.refractions.net
http://postgis.refractions.net/mailman/listinfo/postgis-users