btree_gin: properly call DirectFunctionCall1()

bmomjian · bmomjian · commit 0cf16b44cb74 · 2015-03-31T10:26:45.000-04:00
Previously we called DirectFunctionCall3() with dummy arguments.  Fixed
version of previous patch.

Report by Jon Nelson
diff --git a/contrib/btree_gin/btree_gin.c b/contrib/btree_gin/btree_gin.c
@@ -318,10 +318,7 @@ GIN_SUPPORT(macaddr, false, leftmostvalue_macaddr, macaddr_cmp)
 static Datum
 leftmostvalue_inet(void)
 {
-	return DirectFunctionCall3(inet_in,
-							   CStringGetDatum("0.0.0.0/0"),
-							   ObjectIdGetDatum(0),
-							   Int32GetDatum(-1));
+	return DirectFunctionCall1(inet_in, CStringGetDatum("0.0.0.0/0"));
 }
 GIN_SUPPORT(inet, true, leftmostvalue_inet, network_cmp)
 

Original file line number	Diff line number	Diff line change
`@@ -318,10 +318,7 @@ GIN_SUPPORT(macaddr, false, leftmostvalue_macaddr, macaddr_cmp)`
`318`	`318`	`static Datum`
`319`	`319`	`leftmostvalue_inet(void)`
`320`	`320`	`{`
`321`		`- return DirectFunctionCall3(inet_in,`
`322`		`- CStringGetDatum("0.0.0.0/0"),`
`323`		`- ObjectIdGetDatum(0),`
`324`		`- Int32GetDatum(-1));`
	`321`	`+ return DirectFunctionCall1(inet_in, CStringGetDatum("0.0.0.0/0"));`
`325`	`322`	`}`
`326`	`323`	`GIN_SUPPORT(inet, true, leftmostvalue_inet, network_cmp)`
`327`	`324`