[postgis-devel] [PostGIS] #1427: totopogeom regress failure on 9.1

PostGIS trac at osgeo.org
Tue Jan 3 04:40:46 PST 2012


#1427: totopogeom regress failure on 9.1
----------------------+-----------------------------------------------------
 Reporter:  robe      |       Owner:  strk         
     Type:  defect    |      Status:  new          
 Priority:  medium    |   Milestone:  PostGIS 2.0.0
Component:  topology  |     Version:  trunk        
 Keywords:            |  
----------------------+-----------------------------------------------------

Comment(by robe):

 I usually try to avoid naming the parameters the same as column names to
 avoid this issue (that's one thing I do like better about SQL Server
 TransactSQL is you have to put an @ in front) -- which is why for my user
 functions I always do param_.. so I always avoid the trap. That doesn't
 work too well for a name to expose.

 Anyrate you are violating your own model.  I think your input argument
 should be named
 alayer.  To follow your anedge, atopology (toponame) etc. that is already
 well documented in the docs.

 e.g. like you have here:
 http://www.postgis.org/documentation/manual-svn/ST_AddIsoEdge.html

 and your argument names should agree with what you have documented since
 9.0+ people in theory (why they would I don't know except to be annoyingly
 descriptive), can use named arg calling e.g.


 {{{
 SELECT ST_AddIsoEdge(toponame := 'mytopo', apoint := ST_Point(1,2) ...);
 }}}

-- 
Ticket URL: <http://trac.osgeo.org/postgis/ticket/1427#comment:2>
PostGIS <http://trac.osgeo.org/postgis/>
The PostGIS Trac is used for bug, enhancement & task tracking, a user and developer wiki, and a view into the subversion code repository of PostGIS project.


More information about the postgis-devel mailing list