From 88177f77b17ef478da1dbca9acb5e3a61b346613 Mon Sep 17 00:00:00 2001
From: Tom Lane <tgl@sss.pgh.pa.us>
Date: Mon, 16 Dec 2002 16:22:46 +0000
Subject: Code review for palloc0 patch --- avoid dangerous and unnecessary
 practice of evaluating MemSet's arguments multiple times, except for the
 special case of newNode(), where we can assume the argument is a constant
 sizeof() operator. Also, add GetMemoryChunkContext() to mcxt.c's API, in
 preparation for fixing recent GEQO breakage.

---
 src/backend/nodes/nodes.c | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

(limited to 'src/backend/nodes/nodes.c')

diff --git a/src/backend/nodes/nodes.c b/src/backend/nodes/nodes.c
index 6d1deadb0b..f71bd020ce 100644
--- a/src/backend/nodes/nodes.c
+++ b/src/backend/nodes/nodes.c
@@ -9,7 +9,7 @@
  *
  *
  * IDENTIFICATION
- *	  $Header: /cvsroot/pgsql/src/backend/nodes/nodes.c,v 1.18 2002/11/10 02:17:25 momjian Exp $
+ *	  $Header: /cvsroot/pgsql/src/backend/nodes/nodes.c,v 1.19 2002/12/16 16:22:46 tgl Exp $
  *
  * HISTORY
  *	  Andrew Yu			Oct 20, 1994	file creation
@@ -17,16 +17,11 @@
  *-------------------------------------------------------------------------
  */
 #include "postgres.h"
+
 #include "nodes/nodes.h"
 
 /*
- * newNode -
- *	  create a new node of the specified size and tag the node with the
- *	  specified tag.
- *
- * !WARNING!: Avoid using newNode directly. You should be using the
- *	  macro makeNode. eg. to create a Resdom node, use makeNode(Resdom)
- *
+ * Support for newNode() macro
  */
-Node *newNodeMacroHolder;
 
+Node *newNodeMacroHolder;
-- 
cgit v1.2.1