Add JsonContainerAlloc() macro

Nikita Glukhov · Nikita Glukhov · commit 4079d52e6135 · 2021-03-29T20:40:08.000+03:00
diff --git a/src/backend/utils/adt/json_generic.c b/src/backend/utils/adt/json_generic.c
@@ -173,7 +173,7 @@ JsonValueToContainer(const JsonValue *val)
 		return val->val.binary.data;
 	else
 	{
-		JsonContainerData *jc = palloc(sizeof(JsonContainerData));
+		JsonContainerData *jc = JsonContainerAlloc();
 		jsonvInitContainer(jc, val);
 		return jc;
 	}
diff --git a/src/backend/utils/adt/jsonb_util.c b/src/backend/utils/adt/jsonb_util.c
@@ -641,7 +641,7 @@ fillJsonbValue(const JsonbContainer *container, int index,
 	{
 		Assert(JBE_ISCONTAINER(entry));
 		result->type = jbvBinary;
-		result->val.binary.data = palloc(sizeof(JsonContainerData));
+		result->val.binary.data = JsonContainerAlloc();
 		jsonbInitContainer((JsonContainerData *) result->val.binary.data,
 				/* Remove alignment padding from data pointer and length */
 						   (JsonbContainer *)(base_addr + INTALIGN(offset)),
diff --git a/src/common/jsonapi.c b/src/common/jsonapi.c
@@ -1336,8 +1336,7 @@ jsontFillValue(JsonIterator **pit, JsonValue *res, bool skipNested,
 
 			res->type = jbvBinary;
 			res->val.binary.len = 0;
-			res->val.binary.data = (JsonContainer *)
-							palloc(sizeof(JsonContainer));
+			res->val.binary.data = JsonContainerAlloc();
 
 			if (skipNested)
 			{
diff --git a/src/include/utils/json_generic.h b/src/include/utils/json_generic.h
@@ -249,6 +249,9 @@ JsonIteratorNext(JsonIterator **it, JsonValue *val, bool skipNested)
 
 extern Json *DatumGetJson(Datum value, JsonContainerOps *ops);
 
+#define JsonContainerAlloc() \
+	((JsonContainerData *) palloc(sizeof(JsonContainerData)))
+
 extern JsonValue *JsonFindValueInContainer(JsonContainer *json, uint32 flags,
 										   JsonValue *key);
 

Original file line number	Diff line number	Diff line change
`@@ -173,7 +173,7 @@ JsonValueToContainer(const JsonValue *val)`
`173`	`173`	`return val->val.binary.data;`
`174`	`174`	`else`
`175`	`175`	`{`
`176`		`- JsonContainerData *jc = palloc(sizeof(JsonContainerData));`
	`176`	`+ JsonContainerData *jc = JsonContainerAlloc();`
`177`	`177`	`jsonvInitContainer(jc, val);`
`178`	`178`	`return jc;`
`179`	`179`	`}`
Original file line number	Diff line number	Diff line change
`@@ -641,7 +641,7 @@ fillJsonbValue(const JsonbContainer *container, int index,`
`641`	`641`	`{`
`642`	`642`	`Assert(JBE_ISCONTAINER(entry));`
`643`	`643`	`result->type = jbvBinary;`
`644`		`- result->val.binary.data = palloc(sizeof(JsonContainerData));`
	`644`	`+ result->val.binary.data = JsonContainerAlloc();`
`645`	`645`	`jsonbInitContainer((JsonContainerData *) result->val.binary.data,`
`646`	`646`	`/* Remove alignment padding from data pointer and length */`
`647`	`647`	`(JsonbContainer *)(base_addr + INTALIGN(offset)),`
Original file line number	Diff line number	Diff line change
`@@ -1336,8 +1336,7 @@ jsontFillValue(JsonIterator *pit, JsonValue res, bool skipNested,`
`1336`	`1336`
`1337`	`1337`	`res->type = jbvBinary;`
`1338`	`1338`	`res->val.binary.len = 0;`
`1339`		`- res->val.binary.data = (JsonContainer *)`
`1340`		`- palloc(sizeof(JsonContainer));`
	`1339`	`+ res->val.binary.data = JsonContainerAlloc();`
`1341`	`1340`
`1342`	`1341`	`if (skipNested)`
`1343`	`1342`	`{`