On 04/19/2018 10:01 AM, Marc-André Lureau wrote: > Now that we can safely call QOBJECT() on QObject * as well as its > subtypes, we can have macros qobject_ref() / qobject_unref() that work > everywhere instead of having to use QINCREF() / QDECREF() for QObject > and qobject_incref() / qobject_decref() for its subtypes. > > Note that the new macros evaluate their argument exactly once, thus no > need to shout them. >
It's still useful information to include in the commit message that you did the substitution by sed, then fixed up compiler warnings that resulted from the new macro implementation being slightly more type-safe, as well as fixing up a long line. That way, if someone backports this patch, they know how to resolve conflicts and/or check that they are not missing a conversion. Writing good commit messages is an art form - but in general, if something was questioned during review of a previous revision, then it's never a bad idea to beef up the commit message to answer that question in advance for the next reviewer. > Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com> > Reviewed-by: Eric Blake <ebl...@redhat.com> > --- -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3266 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
