Re: BUG #13905: Inconsistent code modification

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

In response to

Browse pgsql-bugs by date

  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