diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2018-01-19 22:16:25 -0500 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2018-01-19 22:16:25 -0500 |
| commit | 96102a32a374c3b81ba9c2b24bcf1943a87a9ef6 (patch) | |
| tree | f6c2917715868bcba63ae9c8d6df778459f67b74 /src | |
| parent | eee50a8d4c389171ad5180568a7221f7e9b28f09 (diff) | |
| download | postgresql-96102a32a374c3b81ba9c2b24bcf1943a87a9ef6.tar.gz postgresql-96102a32a374c3b81ba9c2b24bcf1943a87a9ef6.zip | |
Suppress possibly-uninitialized-variable warnings.
Apparently, Peter's compiler has faith that the switch test values here
could never not be valid values of their enums. Mine does not, and
I tend to agree with it.
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/catalog/aclchk.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/backend/catalog/aclchk.c b/src/backend/catalog/aclchk.c index de18610a912..1156627b9ec 100644 --- a/src/backend/catalog/aclchk.c +++ b/src/backend/catalog/aclchk.c @@ -3359,7 +3359,7 @@ aclcheck_error(AclResult aclerr, ObjectType objtype, break; case ACLCHECK_NO_PRIV: { - const char *msg; + const char *msg = "???"; switch (objtype) { @@ -3481,7 +3481,6 @@ aclcheck_error(AclResult aclerr, ObjectType objtype, case OBJECT_TSTEMPLATE: case OBJECT_USER_MAPPING: elog(ERROR, "unsupported object type %d", objtype); - msg = "???"; } ereport(ERROR, @@ -3491,7 +3490,7 @@ aclcheck_error(AclResult aclerr, ObjectType objtype, } case ACLCHECK_NOT_OWNER: { - const char *msg; + const char *msg = "???"; switch (objtype) { @@ -3616,7 +3615,6 @@ aclcheck_error(AclResult aclerr, ObjectType objtype, case OBJECT_TSTEMPLATE: case OBJECT_USER_MAPPING: elog(ERROR, "unsupported object type %d", objtype); - msg = "???"; } ereport(ERROR, |