| From: | Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: geo_ops.c: Clarify comments and function arguments |
| Date: | 2018-11-15 15:18:55 |
| Message-ID: | E1gNJPz-0001X2-RY@gemulon.postgresql.org |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
geo_ops.c: Clarify comments and function arguments
These functions were not crystal clear about what their respective APIs
are. Make an effort to improve that.
Emre's patch was correct AFAICT, but I (Álvaro) felt the need to improve
a few comments a bit more. Any resulting errors are my own.
Per complaint from Coverity, Ning Yu, and Tom Lane.
Author: Emre Hasegeli, Álvaro Herrera
Reviewed-by: Tomas Vondra, Álvaro Herrera
Discussion: https://postgr.es/m/26769.1533090136@sss.pgh.pa.us
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/74514bd4a58dcf763d105e60a41a5f1b3eea6865
Modified Files
--------------
src/backend/utils/adt/geo_ops.c | 172 +++++++++++++++++++++++-----------------
1 file changed, 100 insertions(+), 72 deletions(-)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Alvaro Herrera | 2018-11-15 15:20:33 | Re: New Defects reported by Coverity Scan for PostgreSQL |
| Previous Message | Amit Kapila | 2018-11-15 05:43:49 | pgsql: Fix the omission in docs. |