Re: [PATCH v3 00/44] Less clumsy error checking

2020-07-06 Thread Eric Blake
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

Re: [PATCH v3 00/44] Less clumsy error checking

2020-07-06 Thread Markus Armbruster
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,

[PATCH v3 00/44] Less clumsy error checking

2020-07-06 Thread Markus Armbruster
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