Fix comments in heaptuple.c

michaelpq · michaelpq · commit 526b54ece3f6 · 2024-06-28T13:30:47.000+09:00
Since e27f4ee, fastgetattr() and heap_getattr() are not macros, but inlined functions. Author: Junwang Zhao Reviewed-by: Stepan Neretin Discussion: https://postgr.es/m/CAEG8a3JS-JKWWyOcM7BU=vPqFXa3W7mZSHnvc3CBqx=tC+3SCA@mail.gmail.com
diff --git a/src/backend/access/common/heaptuple.c b/src/backend/access/common/heaptuple.c
@@ -495,8 +495,8 @@ heap_attisnull(HeapTuple tup, int attnum, TupleDesc tupleDesc)
 /* ----------------
  *		nocachegetattr
  *
- *		This only gets called from fastgetattr() macro, in cases where
- *		we can't use a cacheoffset and the value is not null.
+ *		This only gets called from fastgetattr(), in cases where we
+ *		can't use a cacheoffset and the value is not null.
  *
  *		This caches attribute offsets in the attribute descriptor.
  *
@@ -715,8 +715,8 @@ nocachegetattr(HeapTuple tup,
  *
  *		Fetch the value of a system attribute for a tuple.
  *
- * This is a support routine for the heap_getattr macro.  The macro
- * has already determined that the attnum refers to a system attribute.
+ * This is a support routine for heap_getattr().  The function has already
+ * determined that the attnum refers to a system attribute.
  * ----------------
  */
 Datum

Original file line number	Diff line number	Diff line change
`@@ -495,8 +495,8 @@ heap_attisnull(HeapTuple tup, int attnum, TupleDesc tupleDesc)`
`495`	`495`	`/* ----------------`
`496`	`496`	`* nocachegetattr`
`497`	`497`	`*`
`498`		`- * This only gets called from fastgetattr() macro, in cases where`
`499`		`- * we can't use a cacheoffset and the value is not null.`
	`498`	`+ * This only gets called from fastgetattr(), in cases where we`
	`499`	`+ * can't use a cacheoffset and the value is not null.`
`500`	`500`	`*`
`501`	`501`	`* This caches attribute offsets in the attribute descriptor.`
`502`	`502`	`*`
`@@ -715,8 +715,8 @@ nocachegetattr(HeapTuple tup,`
`715`	`715`	`*`
`716`	`716`	`* Fetch the value of a system attribute for a tuple.`
`717`	`717`	`*`
`718`		`- * This is a support routine for the heap_getattr macro. The macro`
`719`		`- * has already determined that the attnum refers to a system attribute.`
	`718`	`+ * This is a support routine for heap_getattr(). The function has already`
	`719`	`+ * determined that the attnum refers to a system attribute.`
`720`	`720`	`* ----------------`
`721`	`721`	`*/`
`722`	`722`	`Datum`