Useless assignments

From: Дилян Палаузов <dilyan(dot)palauzov(at)aegee(dot)org>
To: PostgreSQL mailing lists <pgsql-bugs(at)postgresql(dot)org>
Subject: Useless assignments
Date: 2017-06-05 18:21:39
Message-ID: 4f65a18e-b5bc-8a38-7a9a-d3ab6946da98@aegee.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs

diff --git a/src/backend/catalog/partition.c
b/src/backend/catalog/partition.c
--- a/src/backend/catalog/partition.c
+++ b/src/backend/catalog/partition.c
@@ -189,7 +189,6 @@ RelationBuildPartitionDesc(Relation rel)
inhoids = find_inheritance_children(RelationGetRelid(rel), NoLock);

/* Collect bound spec nodes in a list */
- i = 0;
partoids = NIL;
foreach(cell, inhoids)
{
@@ -1508,7 +1508,6 @@ get_qual_for_range(PartitionKey key,
PartitionBoundSpec *spec)

lower_or_start_datum = list_head(spec->lowerdatums);
upper_or_start_datum = list_head(spec->upperdatums);
- num_or_arms = key->partnatts;

/*
* A range-partitioned table does not currently allow partition
keys to be
diff --git a/src/backend/storage/smgr/md.c b/src/backend/storage/smgr/md.c
--- a/src/backend/storage/smgr/md.c
+++ b/src/backend/storage/smgr/md.c
@@ -872,7 +872,7 @@ mdwrite(SMgrRelation reln, ForkNumber forknum,
BlockNumber blocknum,
BlockNumber
mdnblocks(SMgrRelation reln, ForkNumber forknum)
{
- MdfdVec *v = mdopen(reln, forknum, EXTENSION_FAIL);
+ MdfdVec *v; mdopen(reln, forknum, EXTENSION_FAIL);
BlockNumber nblocks;
BlockNumber segno = 0;

diff --git a/src/backend/utils/adt/jsonfuncs.c
b/src/backend/utils/adt/jsonfuncs.c
--- a/src/backend/utils/adt/jsonfuncs.c
+++ b/src/backend/utils/adt/jsonfuncs.c
@@ -4423,7 +4423,7 @@ IteratorConcat(JsonbIterator **it1, JsonbIterator
**it2,
rk2;

r1 = rk1 = JsonbIteratorNext(it1, &v1, false);
- r2 = rk2 = JsonbIteratorNext(it2, &v2, false);
+ rk2 = JsonbIteratorNext(it2, &v2, false);

/*
* Both elements are objects.
diff --git a/src/fe_utils/print.c b/src/fe_utils/print.c
--- a/src/fe_utils/print.c
+++ b/src/fe_utils/print.c
@@ -433,7 +433,6 @@ print_unaligned_text(const printTableContent *cont,
FILE *fout)
if (need_recordsep)
{

print_separator(cont->opt->recordSep, fout);
- need_recordsep = false;
}
fputs(f->data, fout);
need_recordsep = true;
diff --git a/src/backend/replication/logical/worker.c
b/src/backend/replication/logical/worker.c
--- a/src/backend/replication/logical/worker.c
+++ b/src/backend/replication/logical/worker.c
@@ -1202,7 +1202,6 @@ LogicalRepApplyLoop(XLogRecPtr last_received)
if (now >= timeout)
{
requestReply = true;
- ping_sent = true;
}
}
}

Responses

Browse pgsql-bugs by date

  From Date Subject
Next Message Andres Freund 2017-06-05 18:23:45 Re: Useless assignments
Previous Message Robert Haas 2017-06-05 14:36:51 Re: BUG #14682: row level security not work with partitioned table