Re: New Defects reported by Coverity Scan for PostgreSQL
Alvaro Herrera <alvherre@2ndquadrant.com>
From: Alvaro Herrera <alvherre@2ndquadrant.com>
To: Emre Hasegeli <emre@hasegeli.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, Tomas Vondra <tomas.vondra@2ndquadrant.com>, Ning Yu <nyu@pivotal.io>, pgsql-hackers@postgresql.org
Date: 2018-11-15T15:20:33Z
Lists: pgsql-hackers
On 2018-Nov-06, Emre Hasegeli wrote: > > Surely the comment in line 3839 deserves an update :-) > > Done. > > > This seems good material. I would put the detailed conventions comment > > separately from the head of the file, like this (where I also changed > > "Type1 *type1" into "Type1 *obj1", and a few "has" to "have") > > Looks better to me. I found one more "has" and changed it. Pushed, with some further minor changes. I decided not to remove the redundant comments that your patch was removing, as I felt that it's better to keep the API contract together with the function definition. -- Álvaro Herrera https://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
Commits
-
geo_ops.c: Clarify comments and function arguments
- 74514bd4a58d 12.0 landed
-
Refactor geometric functions and operators
- a7dc63d904a6 12.0 cited