add abstract type check here too. (pygobject_init): make this code more

2002-12-24  James Henstridge  <james@daa.com.au>

    * pygobject.c (pygobject_init): add abstract type check here too.
    (pygobject_init): make this code more similar to pyg_object_new,
    so that it is easier to fix bugs in the future.

    * gobjectmodule.c (pyg_object_new): handle case of no keyword
    arguments.
    (pyg_object_new): refuse to instantiate an abstract type.

1 files changed, 46 insertions, 59 deletions

diff --git a/gobject/pygobject.c b/gobject/pygobject.c
index 9a8cb02..76bfb5d 100644
--- a/gobject/pygobject.c
+++ b/gobject/pygobject.c
@@ -339,25 +339,12 @@ pygobject_free(PyObject *op)
 
 /* ---------------- PyGObject methods ----------------- */
 
-static void
-parameter_list_free (GParameter *parameters, guint n_parameters)
-{
-	gint i;
-	for (i = 0; i < n_parameters; i ++){
-		g_free ((char *)parameters[i].name);
-		g_value_unset (&(parameters[i].value));
-	}
-	g_free (parameters);
-}
-
 static int
 pygobject_init(PyGObject *self, PyObject *args, PyObject *kwargs)
 {
     GType object_type;
-    guint n_parameters = 0;
-    GParameter *parameters = NULL;
-    PyObject *key, *item;
-    gint pos = 0;
+    guint n_params = 0, i;
+    GParameter *params = NULL;
     GObjectClass *class;
 
     if (!PyArg_ParseTuple(args, ":GObject.__init__", &object_type))
@@ -367,63 +354,63 @@ pygobject_init(PyGObject *self, PyObject *args, PyObject *kwargs)
     if (!object_type)
 	return -1;
 
+    if (G_TYPE_IS_ABSTRACT(object_type)) {
+	PyErr_Format(PyExc_TypeError, "cannot create instance of abstract "
+		     "(non-instantiable) type `%s'", g_type_name(object_type));
+	return -1;
+    }
+
     if ((class = g_type_class_ref (object_type)) == NULL) {
 	PyErr_SetString(PyExc_TypeError,
 			"could not get a reference to type class");
 	return -1;
     }
 
+    if (kwargs) {
+	int pos = 0;
+	PyObject *key;
+	PyObject *value;
 
-    while (kwargs && PyDict_Next(kwargs, &pos, &key, &item)) {
-	    gchar *param_name;
+	params = g_new0(GParameter, PyDict_Size(kwargs));
+	while (PyDict_Next (kwargs, &pos, &key, &value)) {
 	    GParamSpec *pspec;
-	    GValue value = { 0, };
-
-	    param_name = PyString_AsString(key);
-	    pspec = g_object_class_find_property(class, param_name);
-
-	    if (pspec == NULL) {
-		    gchar buf[128];
-		    
-		    g_snprintf(buf, sizeof(buf),
-			       "Unknown parameter '%s' used in type initialization arguments",
-			       param_name);
-		    PyErr_SetString(PyExc_AttributeError, buf);
-		    parameter_list_free (parameters, n_parameters);
-		    g_type_class_unref(class);
-		    return -1;
-	    }
-	    g_value_init(&value, G_PARAM_SPEC_VALUE_TYPE(pspec));
-	    if (pyg_value_from_pyobject(&value, item) < 0) {
-		    PyErr_SetString(PyExc_TypeError,
-			"could not convert argument to correct param type");
-		    parameter_list_free (parameters, n_parameters);
-		    g_type_class_unref(class);
-		    return -1;
+	    gchar *key_str = PyString_AsString(key);
+
+	    pspec = g_object_class_find_property (class, key_str);
+	    if (!pspec) {
+		PyErr_Format(PyExc_TypeError,
+			     "gobject `%s' doesn't support property `%s'",
+			     g_type_name(object_type), key_str);
+		goto cleanup;
 	    }
-
-	    n_parameters ++;
-	    if (parameters == NULL) {
-		    parameters = g_new (GParameter, 1);
-	    } else {
-		    parameters = g_renew (GParameter, parameters, n_parameters);
+	    g_value_init(&params[n_params].value,
+			 G_PARAM_SPEC_VALUE_TYPE(pspec));
+	    if (pyg_value_from_pyobject(&params[n_params].value, value)) {
+		PyErr_Format(PyExc_TypeError,
+			     "could not convert value for property `%s'",
+			     key_str);
+		goto cleanup;
 	    }
-	    parameters[n_parameters - 1].name = g_strdup (param_name);
-	    parameters[n_parameters - 1].value = value;
+	    params[n_params].name = g_strdup(key_str);
+	    n_params++;
 	}
-
-    self->obj = g_object_newv(object_type, n_parameters, parameters);
-    parameter_list_free (parameters, n_parameters);
-
-    g_type_class_unref(class);
-
-    if (!self->obj) {
-	PyErr_SetString(PyExc_RuntimeError, "could not create object");
-	return -1;
     }
-    pygobject_register_wrapper((PyObject *)self);
 
-    return 0;
+    self->obj = g_object_newv(object_type, n_params, params);
+    if (self->obj)
+	pygobject_register_wrapper((PyObject *)self);
+    else
+	PyErr_SetString (PyExc_RuntimeError, "could not create object");
+	   
+ cleanup:
+    for (i = 0; i < n_params; i++) {
+	g_free((gchar *) params[i].name);
+	g_value_unset(&params[i].value);
+    }
+    g_free(params);
+    g_type_class_unref(class);
+    
+    return (self->obj) ? 0 : -1;
 }
 
 static PyObject *