Re: [libvirt] [PATCH v2 01/15] util: json: add helper to iterate and steal members of json array

Simplifies cases where JSON array members need to be transferred to a different structure. --- Notes: v2: - iterate from beginning - add option to keep array member in the array without breaking iteration - documented callback return codes src/libvirt_private.syms | 1 + src/util/virjson.c | 54 ++++++++++++++++++++++++++++++++++++++++++++++++ src/util/virjson.h | 6 ++++++ 3 files changed, 61 insertions(+) diff --git a/src/libvirt_private.syms b/src/libvirt_private.syms index 162fda5..4c08508 100644 --- a/src/libvirt_private.syms +++ b/src/libvirt_private.syms @@ -1764,6 +1764,7 @@ virISCSIScanTargets; # util/virjson.h virJSONValueArrayAppend; +virJSONValueArrayForeachSteal; virJSONValueArrayGet; virJSONValueArraySize; virJSONValueArraySteal; diff --git a/src/util/virjson.c b/src/util/virjson.c index 1d8e6d5..d55600a 100644 --- a/src/util/virjson.c +++ b/src/util/virjson.c @@ -949,6 +949,60 @@ virJSONValueArraySteal(virJSONValuePtr array, } +/** + * virJSONValueArrayForeachSteal: + * @array: array to iterate + * @cb: callback called on every member of the array + * @opaque: custom data for the callback + * + * Iterates members of the array and calls the callback on every single member. + * The return codes of the callback are interpreted as follows: + * 0: callback claims ownership of the array element and is responsible for + * freeing it

+ * 1: callback does not claim the ownership, but the iteration continues + * -1: callback doesn't claim ownership and iteration does not continue

+ * + * Returns 0 if all members were iterated and/or stolen by the callback; -1 + * on callback failure or if the JSON value object is not an array. + * The rest of the members stay in possession of the array and it's condensed. + */ +int +virJSONValueArrayForeachSteal(virJSONValuePtr array, + virJSONArrayIteratorFunc cb, + void *opaque)