From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | petrum(at)gmail(dot)com |
Cc: | pgsql-bugs(at)postgresql(dot)org |
Subject: | Re: BUG #13905: Inconsistent code modification |
Date: | 2016-02-02 16:18:38 |
Message-ID: | 21244.1454429918@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
petrum(at)gmail(dot)com writes:
> Location: file src/include/catalog/pg_proc.h
> Lines 4613 and 4615 looks to be copies of lines 4252 and 4254,
> respectively.
It's generally better to show the actual lines you're talking about;
line numbers tend to not be all that stable. But I guess you are talking
about the DESCR lines for jsonb_to_record() and jsonb_to_recordset()?
> They are parts of a larger code fragment duplicated and modified by
> replacing *json* with *jsonb*. However, lines 4613 and 4615 are not modified
> accordingly i.e., they still contain the *json* sub-string and not the
> *jsonb* as it would be expected according to their neighbor source code
> lines.
Hmm, yeah. You could argue it either way perhaps as to whether these
comments should refer to JSON or the specific type name, but since all
the other jsonb functions do the latter, these should too.
Thanks for noticing!
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | seth-p | 2016-02-02 19:47:29 | BUG #13908: Query returns too few rows |
Previous Message | marian.krucina | 2016-02-02 16:14:07 | BUG #13907: Restore materialized view throw permission denied |