From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:39948) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UamFu-0003qM-La for qemu-devel@nongnu.org; Fri, 10 May 2013 08:16:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UamFt-0007jp-1j for qemu-devel@nongnu.org; Fri, 10 May 2013 08:16:58 -0400 Received: from mail-wg0-x232.google.com ([2a00:1450:400c:c00::232]:44234) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UamFs-0007jZ-PZ for qemu-devel@nongnu.org; Fri, 10 May 2013 08:16:56 -0400 Received: by mail-wg0-f50.google.com with SMTP id m15so4088970wgh.29 for ; Fri, 10 May 2013 05:16:56 -0700 (PDT) Sender: Paolo Bonzini From: Paolo Bonzini Date: Fri, 10 May 2013 14:16:38 +0200 Message-Id: <1368188203-3407-5-git-send-email-pbonzini@redhat.com> In-Reply-To: <1368188203-3407-1-git-send-email-pbonzini@redhat.com> References: <1368188203-3407-1-git-send-email-pbonzini@redhat.com> Subject: [Qemu-devel] [PATCH for-1.5 4/9] qom: pass file/line/function to asserting casts List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: qemu-devel@nongnu.org Cc: Anthony Liguori , Andreas Faerber , Aurelien Jarno , mst@redhat.com Signed-off-by: Paolo Bonzini --- include/qom/object.h | 16 +++++++++++----- qom/object.c | 15 +++++++++------ 2 files changed, 20 insertions(+), 11 deletions(-) diff --git a/include/qom/object.h b/include/qom/object.h index 41b7068..38f674f 100644 --- a/include/qom/object.h +++ b/include/qom/object.h @@ -476,7 +476,8 @@ struct TypeInfo * generated. */ #define OBJECT_CHECK(type, obj, name) \ - ((type *)object_dynamic_cast_assert(OBJECT(obj), (name))) + ((type *)object_dynamic_cast_assert(OBJECT(obj), (name), \ + __FILE__, __LINE__, __func__)) /** * OBJECT_CLASS_CHECK: @@ -489,7 +490,8 @@ struct TypeInfo * specific class type. */ #define OBJECT_CLASS_CHECK(class, obj, name) \ - ((class *)object_class_dynamic_cast_assert(OBJECT_CLASS(obj), (name))) + ((class *)object_class_dynamic_cast_assert(OBJECT_CLASS(obj), (name), \ + __FILE__, __LINE__, __func__)) /** * OBJECT_GET_CLASS: @@ -547,7 +549,8 @@ struct InterfaceClass * Returns: @obj casted to @interface if cast is valid, otherwise raise error. */ #define INTERFACE_CHECK(interface, obj, name) \ - ((interface *)object_dynamic_cast_assert(OBJECT((obj)), (name))) + ((interface *)object_dynamic_cast_assert(OBJECT((obj)), (name), \ + __FILE__, __LINE__, __func__)) /** * object_new: @@ -615,7 +618,8 @@ Object *object_dynamic_cast(Object *obj, const char *typename); * instead of returning #NULL on failure. This function is not meant to be * called directly, but only through the wrapper macro OBJECT_CHECK. */ -Object *object_dynamic_cast_assert(Object *obj, const char *typename); +Object *object_dynamic_cast_assert(Object *obj, const char *typename, + const char *file, int line, const char *func); /** * object_get_class: @@ -667,7 +671,9 @@ Type type_register(const TypeInfo *info); * OBJECT_CLASS_CHECK and INTERFACE_CHECK. */ ObjectClass *object_class_dynamic_cast_assert(ObjectClass *klass, - const char *typename); + const char *typename, + const char *file, int line, + const char *func); /** * object_class_dynamic_cast: diff --git a/qom/object.c b/qom/object.c index 0aa0c07..bca6219 100644 --- a/qom/object.c +++ b/qom/object.c @@ -431,15 +431,16 @@ Object *object_dynamic_cast(Object *obj, const char *typename) return NULL; } -Object *object_dynamic_cast_assert(Object *obj, const char *typename) +Object *object_dynamic_cast_assert(Object *obj, const char *typename, + const char *file, int line, const char *func) { Object *inst; inst = object_dynamic_cast(obj, typename); if (!inst && obj) { - fprintf(stderr, "Object %p is not an instance of type %s\n", - obj, typename); + fprintf(stderr, "%s:%d:%s: Object %p is not an instance of type %s\n", + file, line, func, obj, typename); abort(); } @@ -495,13 +496,15 @@ ObjectClass *object_class_dynamic_cast(ObjectClass *class, } ObjectClass *object_class_dynamic_cast_assert(ObjectClass *class, - const char *typename) + const char *typename, + const char *file, int line, + const char *func) { ObjectClass *ret = object_class_dynamic_cast(class, typename); if (!ret && class) { - fprintf(stderr, "Object %p is not an instance of type %s\n", - class, typename); + fprintf(stderr, "%s:%d:%s: Object %p is not an instance of type %s\n", + file, line, func, class, typename); abort(); } -- 1.8.1.4