Typos in logical decoding

Started by Antonin Houskaover 9 years ago2 messageshackers
Jump to latest
#1Antonin Houska
ah@cybertec.at

While reading the logical decoding code I noticed a few supposedly mistyped
comments, see the diff below.

Besides that, output_plugin_options argument of CreateInitDecodingContext
function is unused, and the function passes NIL to the corresponding argument
of StartupDecodingContext. This looks to me like a thinko.

--
Antonin Houska
Cybertec Schönig & Schönig GmbH
Gröhrmühlgasse 26
A-2700 Wiener Neustadt
Web: http://www.postgresql-support.de, http://www.cybertec.at

Attachments:

typos.patchtext/x-diffDownload+12-12
#2Magnus Hagander
magnus@hagander.net
In reply to: Antonin Houska (#1)
Re: Typos in logical decoding

On Mon, Jul 18, 2016 at 11:00 AM, Antonin Houska <ah@cybertec.at> wrote:

While reading the logical decoding code I noticed a few supposedly mistyped
comments, see the diff below.

Applied, thanks.

Besides that, output_plugin_options argument of CreateInitDecodingContext
function is unused, and the function passes NIL to the corresponding
argument
of StartupDecodingContext. This looks to me like a thinko.

I agree it looks like a thinko, and AFAICT all internal uses pass NIL there
anyway so it makes no difference on those. I'll wait for someone more
familiar with that code to comment though :)

--
Magnus Hagander
Me: http://www.hagander.net/
Work: http://www.redpill-linpro.com/