From: "D'Arcy J.M. Cain" <darcy@druid.net>

Subject: [PATCHES] backend/lib/fstack.c

  - The FixedStackIsValid so obviously needs to be a macro
  - FixedStackContains only called if assert checking on

diff --git a/src/backend/lib/fstack.c b/src/backend/lib/fstack.c
index b8f90b977adc7116a3160069a2836f91230f6581..68f1505b1673e16e7ecc0dc7e29772d5ff452c8c 100644 (file)
--- a/src/backend/lib/fstack.c
+++ b/src/backend/lib/fstack.c
@@ -7,7 +7,7 @@
  *
  *
  * IDENTIFICATION
- *    $Header: /cvsroot/pgsql/src/backend/lib/Attic/fstack.c,v 1.3 1996/11/06 08:27:12 scrappy Exp $
+ *    $Header: /cvsroot/pgsql/src/backend/lib/Attic/fstack.c,v 1.4 1997/06/06 22:02:37 scrappy Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -39,21 +39,12 @@
 #define FixedStackGetItem(stack, pointer) \
        ((FixedItem)((char *)(pointer) + (stack)->offset))
 
+#define        FixedStackIsValid(stack) ((bool)PointerIsValid(stack))
+
 /*
  * External functions
  */
 
-/*
- * FixedStackIsValid --
- *     True iff stack is valid.
- */
-static bool
-FixedStackIsValid(FixedStack stack)
-{
-    return ((bool)PointerIsValid(stack));
-}
-
-
 void
 FixedStackInit(FixedStack stack, Offset offset)
 {
@@ -92,7 +83,7 @@ FixedStackPush(FixedStack stack, Pointer pointer)
     stack->top = item;
 }
 
-
+#ifndef        NO_ASSERT_CHECKING
 /*
  * FixedStackContains --
  *     True iff ordered stack contains given element.
@@ -122,6 +113,7 @@ FixedStackContains(FixedStack stack, Pointer pointer)
     }
     return (false);
 }
+#endif
 
 Pointer
 FixedStackGetTop(FixedStack stack)