Sync addRangeTableEntryForENR() with its peer functions.
authorTom Lane <tgl@sss.pgh.pa.us>
Sun, 16 Apr 2017 18:02:47 +0000 (14:02 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Sun, 16 Apr 2017 18:02:47 +0000 (14:02 -0400)
commita1888b59b511b42290a6fcfa87e35323d128c4f6
treeac5a8a992618f31b43f9e29d2974b601be1d86d6
parent033b969edde62c84ee15b53972ee638150a28c90
Sync addRangeTableEntryForENR() with its peer functions.

addRangeTableEntryForENR had a check for pstate != NULL, which Coverity
pointed out was rather useless since it'd already dereferenced pstate
before that.  More to the point, we'd established policy in commit
bc93ac12c that we'd require non-NULL pstate for all addRangeTableEntryFor*
functions; this test was evidently copied-and-pasted from some older
version of one of those functions.  Make it look more like the others.

In passing, make an elog message look more like the rest of the code,
too.

Michael Paquier
src/backend/parser/parse_relation.c