diff options
author | Eric Blake <eblake@redhat.com> | 2017-06-24 12:10:08 -0600 |
---|---|---|
committer | Markus Armbruster <armbru@redhat.com> | 2017-07-12 17:51:54 +0200 |
commit | 8a4613a0ab4b6e9271e64aaa88f588ed1b749eb4 (patch) | |
tree | 9e358ef25717802e6e40937ac11ebb91a8355091 /scripts/coccinelle/qobject.cocci | |
parent | 187f47e947d1e0f4547e58b7cc4fad4d5542b185 (diff) |
qobject: Update coccinelle script to catch Q{INC, DEC}REF
The recent commit b097efc0 used qobject_decref(QOBJECT(E)), even
though we already have QDECREF(E) for that purpose. We can update
our coccinelle script to catch any future relapses; with that in
place, the rest of the patch is generated with:
spatch --sp-file scripts/coccinelle/qobject.cocci \
--macro-file scripts/cocci-macro-file.h --dir . --in-place
Signed-off-by: Eric Blake <eblake@redhat.com>
Message-Id: <20170624181008.25497-3-eblake@redhat.com>
Reviewed-by: Stefan Hajnoczi <stefanha@redhat.com>
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Diffstat (limited to 'scripts/coccinelle/qobject.cocci')
-rw-r--r-- | scripts/coccinelle/qobject.cocci | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/scripts/coccinelle/qobject.cocci b/scripts/coccinelle/qobject.cocci index c3253deb1b..c518caf7b1 100644 --- a/scripts/coccinelle/qobject.cocci +++ b/scripts/coccinelle/qobject.cocci @@ -3,6 +3,12 @@ expression Obj, Key, E; @@ ( +- qobject_incref(QOBJECT(E)); ++ QINCREF(E); +| +- qobject_decref(QOBJECT(E)); ++ QDECREF(E); +| - qdict_put_obj(Obj, Key, QOBJECT(E)); + qdict_put(Obj, Key, E); | |