Unused variable in postgres_fdw/deparse.c

Поиск
Список
Период
Сортировка
От Kyotaro HORIGUCHI
Тема Unused variable in postgres_fdw/deparse.c
Дата
Msg-id 20161026.094951.236564575.horiguchi.kyotaro@lab.ntt.co.jp
обсуждение исходный текст
Ответы Re: Unused variable in postgres_fdw/deparse.c  (Ashutosh Bapat <ashutosh.bapat@enterprisedb.com>)
Re: Unused variable in postgres_fdw/deparse.c  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
Hello, compiler complains about unused variable during build
postgres_fdw without assertions (!--enable-cassert).

deparse.c: In function ‘deparseFromExpr’:
deparse.c:1029:14: warning: unused variable ‘foreignrel’ [-Wunused-variable] RelOptInfo *foreignrel =
context->foreignrel;            ^~~~~~~~~~
 

The attched patch removes it and moves into the assertion below it.

regards,

diff --git a/contrib/postgres_fdw/deparse.c b/contrib/postgres_fdw/deparse.c
index 8da8c11..450693a 100644
--- a/contrib/postgres_fdw/deparse.c
+++ b/contrib/postgres_fdw/deparse.c
@@ -1026,11 +1026,10 @@ static voiddeparseFromExpr(List *quals, deparse_expr_cxt *context){    StringInfo    buf =
context->buf;
-    RelOptInfo *foreignrel = context->foreignrel;    RelOptInfo *scanrel = context->scanrel;    /* For upper
relations,scanrel must be either a joinrel or a baserel */
 
-    Assert(foreignrel->reloptkind != RELOPT_UPPER_REL ||
+    Assert(context->foreignrel->reloptkind != RELOPT_UPPER_REL ||           scanrel->reloptkind == RELOPT_JOINREL ||
       scanrel->reloptkind == RELOPT_BASEREL); 

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Noah Misch
Дата:
Сообщение: Re: Wraparound warning
Следующее
От: Amit Langote
Дата:
Сообщение: Re: Declarative partitioning - another take