Commit 40f70de0 authored by Pawel Wodkowski's avatar Pawel Wodkowski Committed by Jim Harris
Browse files

json_util: add debug logs to spdk_json_decode_object function



It is helpful to see what went wrong during decoding.

Change-Id: Ia26d3ea6e97823966840e688eaf9264dfef120a8
Signed-off-by: default avatarPawel Wodkowski <pawelx.wodkowski@intel.com>
Reviewed-on: https://review.gerrithub.io/c/436194


Chandler-Test-Pool: SPDK Automated Test System <sys_sgsw@intel.com>
Tested-by: default avatarSPDK CI Jenkins <sys_sgci@intel.com>
Reviewed-by: default avatarJim Harris <james.r.harris@intel.com>
Reviewed-by: default avatarMaciej Szwed <maciej.szwed@intel.com>
Reviewed-by: default avatarShuhei Matsumoto <shuhei.matsumoto.xt@hitachi.com>
Reviewed-by: default avatarDarek Stojaczyk <dariusz.stojaczyk@intel.com>
parent 122a7763
Loading
Loading
Loading
Loading
+3 −0
Original line number Diff line number Diff line
@@ -331,10 +331,12 @@ spdk_json_decode_object(const struct spdk_json_val *values,
				if (seen[decidx]) {
					/* duplicate field name */
					invalid = true;
					SPDK_JSON_DEBUG("Duplicate key '%s'\n", dec->name);
				} else {
					seen[decidx] = true;
					if (dec->decode_func(v, field)) {
						invalid = true;
						SPDK_JSON_DEBUG("Decoder failed to decode key '%s'\n", dec->name);
						/* keep going to fill out any other valid keys */
					}
				}
@@ -344,6 +346,7 @@ spdk_json_decode_object(const struct spdk_json_val *values,

		if (!found) {
			invalid = true;
			SPDK_JSON_DEBUG("Decoder not found for key '%.*s'\n", name->len, (char *)name->start);
		}

		i += 1 + spdk_json_val_len(v);