Remove JsonValue.val.binary.len field

Author: Nikita Glukhov

src/backend/utils/adt/json_generic.c

@@ -160,7 +160,6 @@ JsonValueWrapInBinary(const JsonValue *val, JsonValue *bin)
 
 	bin->type = jbvBinary;
 	bin->val.binary.data = jc;
-	bin->val.binary.len = jc->len;
 	bin->val.binary.uniquified = JsonValueIsUniquified(val);
 
 	return bin;
@@ -407,7 +406,6 @@ jsonvArrayIteratorNext(JsonIterator **it, JsonValue *res, bool skipNested)
 		{
 			Assert(res->type == jbvArray || res->type == jbvObject);
 			res->val.binary.data = JsonValueToContainer(val);
-			res->val.binary.len = 0;
 			res->val.binary.uniquified = JsonValueIsUniquified(val);
 			res->type = jbvBinary;
 		}
@@ -460,7 +458,6 @@ jsonvObjectIteratorNext(JsonIterator **it, JsonValue *res, bool skipNested)
 			{
 				Assert(res->type == jbvArray || res->type == jbvObject);
 				res->val.binary.data = JsonValueToContainer(&pair->value);
-				res->val.binary.len = 0;
 				res->val.binary.uniquified =
 											JsonValueIsUniquified(&pair->value);
 				res->type = jbvBinary;
@@ -721,7 +718,6 @@ JsonToJsonValue(Json *json, JsonValue *jv)
 
 	jv->type = jbvBinary;
 	jv->val.binary.data = &json->root;
-	jv->val.binary.len = json->root.len;
 	jv->val.binary.uniquified = json->root.ops != &jsontContainerOps;
 
 	return jv;

src/backend/utils/adt/jsonb.c

@@ -608,7 +608,8 @@ JsonbToCStringWorker(StringInfo out, JsonbContainer *in, int estimated_len,
 				{
 					first = false;
 					if (v.type == jbvBinary)
-						JsonbToCString(out, v.val.binary.data, v.val.binary.len);
+						JsonbToCString(out, v.val.binary.data,
+									   v.val.binary.data->len);
 					else
 						jsonb_put_escaped_value(out, &v);
 				}
@@ -634,7 +635,8 @@ JsonbToCStringWorker(StringInfo out, JsonbContainer *in, int estimated_len,
 					add_indent(out, use_indent, level);
 
 				if (v.type == jbvBinary)
-					JsonbToCString(out, v.val.binary.data, v.val.binary.len);
+					JsonbToCString(out, v.val.binary.data,
+								   v.val.binary.data->len);
 				else
 					jsonb_put_escaped_value(out, &v);
 				break;

src/backend/utils/adt/jsonb_util.c

@@ -139,7 +139,6 @@ JsonContainerFlatten(JsonContainer *jc, JsonValueEncoder encoder,
 	{
 		jbv.type = jbvBinary;
 		jbv.val.binary.data = jc;
-		jbv.val.binary.len = jc->len;
 		jbv.val.binary.uniquified = JsonContainerIsUniquified(jc);
 
 		binary = &jbv;
@@ -662,7 +661,6 @@ fillJsonbValue(const JsonbContainer *container, int index,
 						   (JsonbContainer *)(base_addr + INTALIGN(offset)),
 						   getJsonbLength(container, index) -
 								(INTALIGN(offset) - offset));
-		result->val.binary.len = result->val.binary.data->len;
 		result->val.binary.uniquified = true;
 	}
 }

src/backend/utils/adt/jsonfuncs.c

@@ -1845,7 +1845,7 @@ JsonbValueAsText(JsonbValue *v)
 
 				initStringInfo(&jtext);
 				(void) JsonbToCString(&jtext, v->val.binary.data,
-									  v->val.binary.len);
+									  v->val.binary.data->len);
 
 				return cstring_to_text_with_len(jtext.data, jtext.len);
 			}
@@ -3156,7 +3156,7 @@ populate_scalar(ScalarIOData *io, Oid typid, int32 typmod, JsValue *jsv)
 			str = JsonbToCString(NULL, JsonValueToContainer(jbv), 0);
 		else if (jbv->type == jbvBinary)
 			str = JsonbToCString(NULL, jbv->val.binary.data,
-								 jbv->val.binary.len);
+								 jbv->val.binary.data->len);
 		else
 			elog(ERROR, "unrecognized jsonb type: %d", (int) jbv->type);
 	}
@@ -3663,7 +3663,7 @@ populate_record_worker(FunctionCallInfo fcinfo, const char *funcname,
 		/* fill binary jsonb value pointing to jb */
 		jbv.type = jbvBinary;
 		jbv.val.binary.data = &jb->root;
-		jbv.val.binary.len = jb->root.len;
+		jbv.val.binary.uniquified = true;
 	}
 
 	rettuple = populate_composite(&cache->c.io.composite, cache->argtype,

src/backend/utils/adt/jsonpath_exec.c

@@ -2625,7 +2625,6 @@ JsonbInitBinary(JsonbValue *jbv, Jsonb *jb)
 {
 	jbv->type = jbvBinary;
 	jbv->val.binary.data = &jb->root;
-	jbv->val.binary.len = VARSIZE_ANY_EXHDR(jb);
 
 	return jbv;
 }

src/common/jsonapi.c

@@ -1335,7 +1335,6 @@ jsontFillValue(JsonIterator **pit, JsonValue *res, bool skipNested,
 			int		len;
 
 			res->type = jbvBinary;
-			res->val.binary.len = 0;
 			res->val.binary.data = JsonContainerAlloc();
 			res->val.binary.uniquified = false;
 
@@ -1357,8 +1356,6 @@ jsontFillValue(JsonIterator **pit, JsonValue *res, bool skipNested,
 															   : jbvArray,
 							   -1);
 
-			res->val.binary.len = res->val.binary.data->len;
-
 			if (skipNested)
 				return false;
 

src/include/utils/jsonb.h

@@ -307,7 +307,6 @@ struct JsonbValue
 
 		struct
 		{
-			int			len; /* FIXME remove */
 			bool		uniquified;
 			const struct JsonContainerData *data;
 		}			binary;		/* Array or object, in on-disk format */