| From: | Emre Hasegeli <emre(at)hasegeli(dot)com> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Ning Yu <nyu(at)pivotal(dot)io>, Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Tomas Vondra <tomas(dot)vondra(at)2ndquadrant(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: New Defects reported by Coverity Scan for PostgreSQL |
| Date: | 2018-08-01 09:55:53 |
| Message-ID: | CAE2gYzw6iD1YTZ+67hA=s3AG5sbLCsGkHRYQ_Z1Kd+vhiM7ecA@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers pgsql-hackers |
> Or perhaps I have it backwards and "l1" and "l2" need to be swapped in
> that description. But the mere fact that there is any question about
> that means that the function is poorly documented and perhaps poorly
> named as well. For that matter, is there a good reason why l1/l2
> have those roles and not the reverse?
Consistency. I organized all xxx_closept_yyy(Point *result, xxx *l1,
yyy *l2) functions in a way that they find the find the point on "l1".
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2018-08-01 12:15:27 | pgsql: Update parallel.sgml for Parallel Append |
| Previous Message | Tomas Vondra | 2018-08-01 09:23:16 | Re: New Defects reported by Coverity Scan for PostgreSQL |
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Sergei Kornilov | 2018-08-01 10:16:58 | Re: Online enabling of checksums |
| Previous Message | Tomas Vondra | 2018-08-01 09:23:16 | Re: New Defects reported by Coverity Scan for PostgreSQL |