On Mon, Apr 20, 2015 at 05:04:14PM -0400, Robert Haas wrote:
> On Mon, Apr 20, 2015 at 4:11 PM, Bruce Momjian <br...@momjian.us> wrote:
> > Slightly improved patch applied.
> 
> Is it?

The patch has a slightly modified 'if' statement to check a constant
before calling a function, and use elseif:

        < +     if (!interpretOidsOption(stmt->options, true) && cxt.hasoids)
        ---
        > +     if (cxt.hasoids && !interpretOidsOption(stmt->options, true))
        47c57
        < +     if (interpretOidsOption(stmt->options, true) && !cxt.hasoids)
        ---
        > +     else if (!cxt.hasoids && interpretOidsOption(stmt->options, 
true))

I realize the change is subtle.

-- 
  Bruce Momjian  <br...@momjian.us>        http://momjian.us
  EnterpriseDB                             http://enterprisedb.com

  + Everyone has their own god. +


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to