Cleanup in contrib/postgres_fdw/deparse.c

Started by Etsuro Fujitaover 9 years ago3 messages
#1Etsuro Fujita
fujita.etsuro@lab.ntt.co.jp
1 attachment(s)

Hi,

Here is a small patch to remove an unnecessary return from
deparseFromExprForRel in contrib/postgres_fdw/deparse.c.

Best regards,
Etsuro Fujita

Attachments:

pg-fdw-deparse-cleanup.patchapplication/x-download; name=pg-fdw-deparse-cleanup.patchDownload
*** a/contrib/postgres_fdw/deparse.c
--- b/contrib/postgres_fdw/deparse.c
***************
*** 1210,1216 **** deparseFromExprForRel(StringInfo buf, PlannerInfo *root, RelOptInfo *foreignrel,
  
  		heap_close(rel, NoLock);
  	}
- 	return;
  }
  
  /*
--- 1210,1215 ----
#2Robert Haas
robertmhaas@gmail.com
In reply to: Etsuro Fujita (#1)
Re: Cleanup in contrib/postgres_fdw/deparse.c

On Mon, Jun 20, 2016 at 4:47 AM, Etsuro Fujita
<fujita.etsuro@lab.ntt.co.jp> wrote:

Here is a small patch to remove an unnecessary return from
deparseFromExprForRel in contrib/postgres_fdw/deparse.c.

Seems like a good cleanup. Committed.

--
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

#3Etsuro Fujita
fujita.etsuro@lab.ntt.co.jp
In reply to: Robert Haas (#2)
Re: Cleanup in contrib/postgres_fdw/deparse.c

On 2016/06/25 3:39, Robert Haas wrote:

Seems like a good cleanup. Committed.

Thanks for committing the patch!

Best regards,
Etsuro Fujita

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