Commit ea5ef5c8 authored by Markus Armbruster's avatar Markus Armbruster
Browse files

qjson: Abort earlier on qobject_from_jsonf() misuse 



Ignoring errors first, then asserting success is suboptimal.  Pass
&error_abort instead, so we abort earlier, and hopefully get more
useful clues on what's wrong.

Signed-off-by: default avatarMarkus Armbruster <armbru@redhat.com>
Reviewed-by: default avatarKevin Wolf <kwolf@redhat.com>
Message-Id: <1488317230-26248-11-git-send-email-armbru@redhat.com>
parent 53f99152

qobject/qjson.c

+1 −1
Original line number Diff line number Diff line
@@ -65,7 +65,7 @@ QObject *qobject_from_jsonf(const char *string, ...) 
    va_list ap;



    va_start(ap, string);

    obj = qobject_from_jsonv(string, &ap, NULL);

    obj = qobject_from_jsonv(string, &ap, &error_abort);

    va_end(ap);



    assert(obj != NULL);