> Tatsuo Ishii wrote: > > ! if (!PortalIsValid(portal) || (PortalIsValid(portal) && portal->cleanup > > == NULL)) > > > > > > > Surely the second call to PortalIsValid() is redundant. > > if (( !PortalIsValid(portal)) || portal->cleanup == NULL) > > should do it, no?
Oops. You are right. -- Tatsuo Ishii SRA OSS, Inc. Japan -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
