From 0a3cee72ac0cd0ca695395b92cd74ff026da8e6a Mon Sep 17 00:00:00 2001
From: Bruce Momjian <bruce@momjian.us>
Date: Sun, 14 Sep 1997 03:59:44 +0000
Subject: [PATCH] Add mention of heap_getattr() compiler warning.

---
 src/include/access/heapam.h | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/include/access/heapam.h b/src/include/access/heapam.h
index c5759c6e76..7236ce92a7 100644
--- a/src/include/access/heapam.h
+++ b/src/include/access/heapam.h
@@ -6,7 +6,7 @@
  *
  * Copyright (c) 1994, Regents of the University of California
  *
- * $Id: heapam.h,v 1.18 1997/09/12 14:29:04 momjian Exp $
+ * $Id: heapam.h,v 1.19 1997/09/14 03:59:44 momjian Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -93,6 +93,10 @@ typedef HeapAccessStatisticsData *HeapAccessStatistics;
  *		<tup> is the pointer to the heap tuple.  <attnum> is the attribute
  *		number of the column (field) caller wants.	<tupleDesc> is a
  *		pointer to the structure describing the row and all its fields.
+ *
+ *		Because this macro is often called with constants, it generates
+ *		compiler warnings about 'left-hand comma expression has no effect.
+ *
  * ---------------- */
 #define heap_getattr(tup, b, attnum, tupleDesc, isnull) \
 	(AssertMacro((tup) != NULL) ? \