diff options
| author | Gustavo J. A. M. Carneiro <gjc@src.gnome.org> | 2006-07-29 13:31:16 +0000 |
|---|---|---|
| committer | Gustavo J. A. M. Carneiro <gjc@src.gnome.org> | 2006-07-29 13:31:16 +0000 |
| commit | 2ec5c81e5ad177b665d6a915aa814663f75996c0 (patch) | |
| tree | 397dbf7e16dc81cd3b7bce9c5b4de40339dfaeb5 | |
| parent | 263a8954ef973fb94aafa7c0df039681dbb12d1f (diff) | |
| download | pygobject-2ec5c81e5ad177b665d6a915aa814663f75996c0.tar.gz pygobject-2ec5c81e5ad177b665d6a915aa814663f75996c0.tar.xz pygobject-2ec5c81e5ad177b665d6a915aa814663f75996c0.zip | |
better debug warnings
| -rw-r--r-- | ChangeLog | 7 | ||||
| -rw-r--r-- | gobject/pygenum.c | 6 | ||||
| -rw-r--r-- | gobject/pygflags.c | 6 |
3 files changed, 17 insertions, 2 deletions
@@ -1,3 +1,10 @@ +2006-07-29 Gustavo J. A. M. Carneiro <gjc@gnome.org> + + * gobject/pygflags.c (pyg_flags_add): idem. + + * gobject/pygenum.c (pyg_enum_add): Give a more explicit error + message when the gtype is not an enum. + 2006-07-25 Johan Dahlin <jdahlin@async.com.br> * gobject/pygobject.h (init_pygobject_check): Add a macro to easily diff --git a/gobject/pygenum.c b/gobject/pygenum.c index d1bb20f..305dd68 100644 --- a/gobject/pygenum.c +++ b/gobject/pygenum.c @@ -174,7 +174,11 @@ pyg_enum_add (PyObject * module, int i; g_return_val_if_fail(typename != NULL, NULL); - g_return_val_if_fail(g_type_is_a(gtype, G_TYPE_ENUM), NULL); + if (!g_type_is_a(gtype, G_TYPE_ENUM)) { + g_warning("Trying to register gtype '%s' as enum when in fact it is of type '%s'", + g_type_name(gtype), g_type_name(G_TYPE_FUNDAMENTAL(gtype))); + return NULL; + } state = pyg_gil_state_ensure(); diff --git a/gobject/pygflags.c b/gobject/pygflags.c index 442e332..bfd5128 100644 --- a/gobject/pygflags.c +++ b/gobject/pygflags.c @@ -202,7 +202,11 @@ pyg_flags_add (PyObject * module, int i; g_return_val_if_fail(typename != NULL, NULL); - g_return_val_if_fail(g_type_is_a(gtype, G_TYPE_FLAGS), NULL); + if (!g_type_is_a(gtype, G_TYPE_FLAGS)) { + g_warning("Trying to register gtype '%s' as flags when in fact it is of type '%s'", + g_type_name(gtype), g_type_name(G_TYPE_FUNDAMENTAL(gtype))); + return NULL; + } state = pyg_gil_state_ensure(); |
