From 0aeb5ae2041520f02cabbc7083aec46733689bce Mon Sep 17 00:00:00 2001 From: Alvaro Herrera Date: Mon, 22 Jul 2013 13:15:13 -0400 Subject: [PATCH] Silence compiler warning on an unused variable Also, tweak wording in comments (per Andres) and documentation (myself) to point out that it's the database's default tablespace that can be passed as 0, not DEFAULTTABLESPACE_OID. Robert Haas noticed the bug in the code, but didn't update the accompanying prose. --- doc/src/sgml/func.sgml | 4 ++-- src/backend/utils/adt/dbsize.c | 7 ++++--- src/backend/utils/cache/relfilenodemap.c | 2 +- 3 files changed, 7 insertions(+), 6 deletions(-) diff --git a/doc/src/sgml/func.sgml b/doc/src/sgml/func.sgml index da63b347e5..039b279f78 100644 --- a/doc/src/sgml/func.sgml +++ b/doc/src/sgml/func.sgml @@ -15814,8 +15814,8 @@ postgres=# SELECT * FROM pg_xlogfile_name_offset(pg_stop_backup()); pg_filenode_relation is the reverse of pg_relation_filenode. Given a tablespace OID and - a filenode it returns the associated relation. The default - tablespace can be specified as 0. + a filenode, it returns the associated relation's OID. For a table + in the database's default tablespace, the tablespace can be specified as 0. diff --git a/src/backend/utils/adt/dbsize.c b/src/backend/utils/adt/dbsize.c index 21d1c946ab..868474680d 100644 --- a/src/backend/utils/adt/dbsize.c +++ b/src/backend/utils/adt/dbsize.c @@ -760,13 +760,14 @@ pg_relation_filenode(PG_FUNCTION_ARGS) * Get the relation via (reltablespace, relfilenode) * * This is expected to be used when somebody wants to match an individual file - * on the filesystem back to its table. Thats not trivially possible via - * pg_class because that doesn't contain the relfilenodes of shared and nailed + * on the filesystem back to its table. That's not trivially possible via + * pg_class, because that doesn't contain the relfilenodes of shared and nailed * tables. * * We don't fail but return NULL if we cannot find a mapping. * - * Instead of knowing DEFAULTTABLESPACE_OID you can pass 0. + * InvalidOid can be passed instead of the current database's default + * tablespace. */ Datum pg_filenode_relation(PG_FUNCTION_ARGS) diff --git a/src/backend/utils/cache/relfilenodemap.c b/src/backend/utils/cache/relfilenodemap.c index 372cb33c00..2a8f83775d 100644 --- a/src/backend/utils/cache/relfilenodemap.c +++ b/src/backend/utils/cache/relfilenodemap.c @@ -210,7 +210,7 @@ RelidByRelfilenode(Oid reltablespace, Oid relfilenode) while (HeapTupleIsValid(ntp = systable_getnext(scandesc))) { - bool isnull; + bool isnull PG_USED_FOR_ASSERTS_ONLY; if (found) elog(ERROR,