<html>
<head>
<title></title>
<meta name="GENERATOR" content="MSHTML 8.00.6001.18854"></meta>
</head>
<body>fixed at r5149
<div align="left"> </div>
<div align="left">I'm learning by doing :-)<br />
</div>
<div align="left">Thanks</div>
<div align="left">Nicklas</div>
<div align="left"> </div>
<div align="left"><br />
2010-01-21 Mark Cave-Ayland wrote:<br />
<br />
PostGIS wrote:<br />
><br />
>> Comment:<br />
>> <br />
>> OK, committed at r5148.[[BR]]<br />
>> <br />
>> My first ever commit to the trunk :-)<br />
><br />
>Congratulations! :)<br />
><br />
>A couple of minor points I've just picked up looking at the patch:<br />
><br />
>- Can you please re-instate the comment included in your original <br />
>version - otherwise in several years we'll be wondering why on earth <br />
>this hack exists in the first place ;)<br />
><br />
>- Please maintain the line spacing precedent set by the existing file, <br />
>e.g. 2 new lines between each commented major section. It just makes it <br />
>easier on the eye trying to locate specific sections.<br />
><br />
><br />
>ATB,<br />
><br />
>Mark.<br />
><br />
>-- <br />
>Mark Cave-Ayland - Senior Technical Architect<br />
>PostgreSQL - PostGIS<br />
>Sirius Corporation plc - control through freedom<br />
>http://www.siriusit.co.uk<br />
>t: +44 870 608 0063<br />
><br />
>Sirius Labs: http://www.siriusit.co.uk/labs<br />
>_______________________________________________<br />
>postgis-devel mailing list<br />
>postgis-devel@postgis.refractions.net<br />
>http://postgis.refractions.net/mailman/listinfo/postgis-devel<br />
><br />
></div>
</body>
</html>