diff options
| author | Tom Lane <tgl@sss.pgh.pa.us> | 2000-06-14 05:24:50 +0000 |
|---|---|---|
| committer | Tom Lane <tgl@sss.pgh.pa.us> | 2000-06-14 05:24:50 +0000 |
| commit | ff7b9f55410bbfbd017af331d23371ab61b63d5e (patch) | |
| tree | 6a055be48dd161470503eb2f1a0376bc048ab73b /src/backend/access/nbtree/nbtree.c | |
| parent | 69cd08d9f79818c8ff8b7b74c7ecfe8dcdb0202c (diff) | |
| download | postgresql-ff7b9f55410bbfbd017af331d23371ab61b63d5e.tar.gz postgresql-ff7b9f55410bbfbd017af331d23371ab61b63d5e.zip | |
I had overlooked the fact that some fmgr-callable functions return void
--- ie, they're only called for side-effects. Add a PG_RETURN_VOID()
macro and use it where appropriate. This probably doesn't change the
machine code by a single bit ... it's just for documentation.
Diffstat (limited to 'src/backend/access/nbtree/nbtree.c')
| -rw-r--r-- | src/backend/access/nbtree/nbtree.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/src/backend/access/nbtree/nbtree.c b/src/backend/access/nbtree/nbtree.c index ccd5a265d31..7dbc08fbfc8 100644 --- a/src/backend/access/nbtree/nbtree.c +++ b/src/backend/access/nbtree/nbtree.c @@ -12,7 +12,7 @@ * Portions Copyright (c) 1994, Regents of the University of California * * IDENTIFICATION - * $Header: /cvsroot/pgsql/src/backend/access/nbtree/nbtree.c,v 1.56 2000/06/13 07:34:38 tgl Exp $ + * $Header: /cvsroot/pgsql/src/backend/access/nbtree/nbtree.c,v 1.57 2000/06/14 05:24:37 tgl Exp $ * *------------------------------------------------------------------------- */ @@ -336,7 +336,7 @@ btbuild(PG_FUNCTION_ARGS) /* all done */ BuildingBtree = false; - PG_RETURN_POINTER(NULL); /* no real return value */ + PG_RETURN_VOID(); } /* @@ -502,7 +502,7 @@ btrescan(PG_FUNCTION_ARGS) so->numberOfKeys * sizeof(ScanKeyData)); } - PG_RETURN_POINTER(NULL); /* no real return value */ + PG_RETURN_VOID(); } void @@ -560,7 +560,7 @@ btendscan(PG_FUNCTION_ARGS) _bt_dropscan(scan); - PG_RETURN_POINTER(NULL); /* no real return value */ + PG_RETURN_VOID(); } /* @@ -592,7 +592,7 @@ btmarkpos(PG_FUNCTION_ARGS) so->mrkHeapIptr = so->curHeapIptr; } - PG_RETURN_POINTER(NULL); /* no real return value */ + PG_RETURN_VOID(); } /* @@ -625,7 +625,7 @@ btrestrpos(PG_FUNCTION_ARGS) so->curHeapIptr = so->mrkHeapIptr; } - PG_RETURN_POINTER(NULL); /* no real return value */ + PG_RETURN_VOID(); } /* stubs */ @@ -641,7 +641,7 @@ btdelete(PG_FUNCTION_ARGS) /* delete the data from the page */ _bt_pagedel(rel, tid); - PG_RETURN_POINTER(NULL); /* no real return value */ + PG_RETURN_VOID(); } static void |