On 9/24/19 8:28 AM, Markus Armbruster wrote: > Point to the previous definition, unless it's a built-in. > > Signed-off-by: Markus Armbruster <arm...@redhat.com> > ---
> if other_ent: > + if other_ent.info: > + where = QAPIError(other_ent.info, None, "previous > definition") > + raise QAPISemError( > + ent.info, > + "'%s' is already defined\n%s" % (ent.name, where)) > raise QAPISemError( > ent.info, "%s is already defined" % other_ent.describe()) Nice. Reviewed-by: Eric Blake <ebl...@redhat.com> -- Eric Blake, Principal Software Engineer Red Hat, Inc. +1-919-301-3226 Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
