On 7/6/20 3:12 AM, Markus Armbruster wrote:
diff between v2 rebased and v3, with hunks that change only whitespace
dropped:
Thanks, that's useful.
Overall, the series looks ready to go from my perspective. It looks
like the changes from v2 were minimal enough that I don't have any R-b
to
diff between v2 rebased and v3, with hunks that change only whitespace
dropped:
diff --git a/qapi/opts-visitor.c b/qapi/opts-visitor.c
index 6d8f4b6928..7781c23a42 100644
--- a/qapi/opts-visitor.c
+++ b/qapi/opts-visitor.c
@@ -235,12 +235,12 @@ opts_start_list(Visitor *v, const char *name,
When the Error API was created, we adopted the (unwritten) rule to
return void when the function returns no useful value on success,
unlike GError, which recommends to return true on success and false on
error then.
When a function returns a distinct error value, say false, a checked
call that