On Wed, Feb 4, 2015 at 7:00 AM, Michael Paquier <michael.paqu...@gmail.com> wrote: > Robert Haas wrote: >> I wrote: >>> src, dst and ctx are created respectively from mbuf_create_from_data, >>> mbuf_create and pgp_init which never return NULL and they are palloc'd >>> all the time. I think that we could simplify things with the patch >>> attached, note that I added an assertion for correctness but I don't >>> really think that it is much necessary. >> >> Yeah, I'd drop the assertion. Also, how about changing things around >> slightly so that we lose the goto-label construct? There's only one >> goto, and its only about 6 lines before the label, so we could just >> flip the sense of the if-test and put the code that gets skipped >> inside the if-block. > Good idea. This gives the patch attached then.
Committed. (BTW, why do you not leave a blank line between quoted text and your responses?) -- Robert Haas EnterpriseDB: http://www.enterprisedb.com The Enterprise PostgreSQL Company -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers