Fix a minor bug in outfuncs support for SetOp: dupOperators is an array

Neil Conway · Neil Conway · commit 6d389bfd26b2 · 2008-01-07T21:33:10.000Z
of Oid, and therefore should use the "%u" escape sequence rather than "%d".
diff --git a/src/backend/nodes/outfuncs.c b/src/backend/nodes/outfuncs.c
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *	  $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.320 2008/01/01 19:45:50 momjian Exp $
+ *	  $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.321 2008/01/07 21:33:10 neilc Exp $
  *
  * NOTES
  *	  Every node type that can appear in stored rules' parsetrees *must*
@@ -600,7 +600,7 @@ _outSetOp(StringInfo str, SetOp *node)
 
 	appendStringInfo(str, " :dupOperators");
 	for (i = 0; i < node->numCols; i++)
-		appendStringInfo(str, " %d", node->dupOperators[i]);
+		appendStringInfo(str, " %u", node->dupOperators[i]);
 
 	WRITE_INT_FIELD(flagColIdx);
 }

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@`
`8`	`8`	`*`
`9`	`9`	`*`
`10`	`10`	`* IDENTIFICATION`
`11`		`- * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.320 2008/01/01 19:45:50 momjian Exp $`
	`11`	`+ * $PostgreSQL: pgsql/src/backend/nodes/outfuncs.c,v 1.321 2008/01/07 21:33:10 neilc Exp $`
`12`	`12`	`*`
`13`	`13`	`* NOTES`
`14`	`14`	`* Every node type that can appear in stored rules' parsetrees must`
`@@ -600,7 +600,7 @@ _outSetOp(StringInfo str, SetOp *node)`
`600`	`600`
`601`	`601`	`appendStringInfo(str, " :dupOperators");`
`602`	`602`	`for (i = 0; i < node->numCols; i++)`
`603`		`- appendStringInfo(str, " %d", node->dupOperators[i]);`
	`603`	`+ appendStringInfo(str, " %u", node->dupOperators[i]);`
`604`	`604`
`605`	`605`	`WRITE_INT_FIELD(flagColIdx);`
`606`	`606`	`}`