Re: New Defects reported by Coverity Scan for PostgreSQL - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: New Defects reported by Coverity Scan for PostgreSQL
Date
Msg-id 20181106173210.46xq6mcsp3b6a2h7@alvherre.pgsql
Whole thread Raw
In response to Re: New Defects reported by Coverity Scan for PostgreSQL  (Emre Hasegeli <emre@hasegeli.com>)
Responses Re: New Defects reported by Coverity Scan for PostgreSQL
List pgsql-hackers
On 2018-Nov-06, Emre Hasegeli wrote:

> The patch is attached to improve the comments and variable names.  I
> explained the functions with the same signature on the file header.  I
> can duplicate those on the function headers if you find that better.

Surely the comment in line 3839 deserves an update :-)

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")

Thanks

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services

Attachment

pgsql-hackers by date:

Previous
From: "REIX, Tony"
Date:
Subject: RE: Issue with v11.0 within jsonb_plperl tests in 32bit on AIX
Next
From: Andres Freund
Date:
Subject: Re: valgrind issues on Fedora 28