<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Oct 7, 2021 at 6:43 PM Ján Tomko <<a href="mailto:jtomko@redhat.com">jtomko@redhat.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On a Thursday in 2021, Peter Krempa wrote:<br>
>In many cases we use a signed value, but use the sign to note that it<br>
>was not assigned. For converting to JSON objects it will be handy to<br>
>have possibility to do this automatically.<br>
><br>
>Signed-off-by: Peter Krempa <<a href="mailto:pkrempa@redhat.com" target="_blank">pkrempa@redhat.com</a>><br>
>---<br>
> src/util/virjson.c | 7 ++++++-<br>
> 1 file changed, 6 insertions(+), 1 deletion(-)<br>
><br>
>diff --git a/src/util/virjson.c b/src/util/virjson.c<br>
>index 9adcea4fff..70ea71b505 100644<br>
>--- a/src/util/virjson.c<br>
>+++ b/src/util/virjson.c<br>
>@@ -115,6 +115,7 @@ virJSONValueGetType(const virJSONValue *value)<br>
>  *<br>
>  * i: signed integer value<br>
>  * j: signed integer value, error if negative<br>
>+ * k: signed integer value, omitted if negative<br>
>  * z: signed integer value, omitted if zero<br>
>  * y: signed integer value, omitted if zero, error if negative<br>
>  *<br>
>@@ -189,6 +190,7 @@ virJSONValueObjectAddVArgs(virJSONValue *obj,<br>
><br>
>         case 'z':<br>
>         case 'y':<br>
>+        case 'k':<br>
>         case 'j':<br>
>         case 'i': {<br>
>             int val = va_arg(args, int);<br>
>@@ -200,7 +202,10 @@ virJSONValueObjectAddVArgs(virJSONValue *obj,<br>
>                 return -1;<br>
>             }<br>
><br>
>-            if (!val && (type == 'z' || type == 'y'))<br>
>+            if (val == 0 && (type == 'z' || type == 'y'))<br>
>+                continue;<br>
>+<br>
<br>
Please split out this cosmetic style change.<br></blockquote><div><br></div><div>Please don't. <br></div><div> <br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
>+            if (val < 0 && type == 'k')<br>
>                 continue;<br>
><br>
<br>
k.<br>
<br>
I mean<br>
Reviewed-by: Ján Tomko <<a href="mailto:jtomko@redhat.com" target="_blank">jtomko@redhat.com</a>><br>
<br>
Jano<br></blockquote><div><br></div><div>Kristina <br></div></div></div>