<HTML dir=ltr><HEAD><TITLE>[postgis-devel] Issue 35 in postgis: Comments on Postgis Functions</TITLE>
<META http-equiv=Content-Type content="text/html; charset=unicode">
<META content="MSHTML 6.00.6000.16674" name=GENERATOR></HEAD>
<BODY>
<DIV id=idOWAReplyText16337 dir=ltr>
<DIV dir=ltr><FONT face=Arial color=#000000 size=2>Paul,</FONT></DIV>
<DIV dir=ltr><FONT face=Arial color=#000000 size=2>You must have been reading my mind when you selectively picked this one and the proj one to not comment about.</FONT></DIV>
<DIV dir=ltr><FONT face=Arial size=2></FONT> </DIV>
<DIV dir=ltr><FONT face=Arial size=2>Regarding this I have been looking at Dane's submission and as Paul mentioned, off list, its not good to add any more dependencies than we need to.  This would require adding Python and Python's pgsql library to use his source.</FONT></DIV>
<DIV dir=ltr><FONT face=Arial size=2></FONT> </DIV>
<DIV dir=ltr><FONT face=Arial size=2>My main gripe with it is that it requires a postgis_template db to be loaded to build the comments.  It seems the main reason for that is to look up the arg signature in the procname which would seem to be unnecessary if our argument list in the documents could be mapped to the postgresql function signature.</FONT></DIV>
<DIV dir=ltr><FONT face=Arial size=2></FONT> </DIV>
<DIV dir=ltr><FONT face=Arial size=2>In many cases they do or almost do.  But we have cases like</FONT></DIV>
<DIV dir=ltr><FONT face=Arial size=2></FONT> </DIV>
<DIV dir=ltr><SPAN class=term>ST_MakePolygon(linestring, [linestring[]])</SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term>which if we changed to</SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term>ST_MakePolygon(linestring geometry, linestrings geometry[])</SPAN></DIV>
<DIV dir=ltr><SPAN class=term><SPAN class=term><SPAN class=term>ST_MakePolygon(linestring)</SPAN></SPAN></SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term>would work.</SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term>the only issue I can think of are these functions with multi-signatures</SPAN></DIV>
<DIV dir=ltr><SPAN class=term><SPAN class=term>ST_AddPoint(linestring, point, [<position>])</SPAN></SPAN></DIV>
<DIV dir=ltr><SPAN class=term><SPAN class=term></SPAN></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term><SPAN class=term>I personally would prefer see it listed twice as</SPAN></SPAN></DIV>
<DIV dir=ltr><SPAN class=term><SPAN class=term><SPAN class=term>ST_AddPoint(linestring geometry, point geometry)</SPAN></SPAN></SPAN></DIV>
<DIV dir=ltr><SPAN class=term><SPAN class=term><SPAN class=term></SPAN></SPAN></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term><SPAN class=term><SPAN class=term><SPAN class=term>ST_AddPoint(linestring geometry, point geometry, position integer)</SPAN></SPAN></SPAN></SPAN></DIV>
<DIV dir=ltr><SPAN class=term><SPAN class=term><SPAN class=term><SPAN class=term></SPAN></SPAN></SPAN></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term><SPAN class=term><SPAN class=term><SPAN class=term>since I'm not convinced non-programmers don't find that nomenclature confusing.</SPAN></SPAN></SPAN></SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term>then I'm thinking we can generate the postgis_comments.sql.in from a simple</SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term>postgis_comments.xsl  file that parses the postgis.xml thus not needing another dependency.</SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term>Although I could very well be trivializing the simplicity of this.</SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term>Does anyone have issues with me changing the doc accordingly.  </SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term>Thanks,</SPAN></DIV>
<DIV dir=ltr><SPAN class=term>Regina</SPAN></DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV>
<DIV dir=ltr><SPAN class=term></SPAN> </DIV></DIV>
<DIV dir=ltr><BR>
<HR tabIndex=-1>
<FONT face=Tahoma size=2><B>From:</B> postgis-devel-bounces@postgis.refractions.net on behalf of codesite-noreply@google.com<BR><B>Sent:</B> Thu 7/3/2008 5:47 PM<BR><B>To:</B> postgis-devel@postgis.refractions.net<BR><B>Subject:</B> [postgis-devel] Issue 35 in postgis: Comments on Postgis Functions<BR></FONT><BR></DIV>
<DIV>
<P><FONT size=2>Issue 35: Comments on Postgis Functions<BR><A href="http://code.google.com/p/postgis/issues/detail?id=35">http://code.google.com/p/postgis/issues/detail?id=35</A><BR><BR>Comment #2 by pwramsey3:<BR>(No comment was entered for this change.)<BR><BR><BR>Issue attribute updates:<BR>        Labels: -Type-Defect Type-Enhancement<BR><BR>--<BR>You received this message because you are listed in the owner<BR>or CC fields of this issue, or because you starred this issue.<BR>You may adjust your issue notification preferences at:<BR><A href="http://code.google.com/hosting/settings">http://code.google.com/hosting/settings</A><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></FONT></P></DIV></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>