guint64 property fix

2 files changed, 12 insertions, 11 deletions

diff --git a/gobject/pygtype.c b/gobject/pygtype.c
index 9b4dcda6..ebb9dd8c 100644
--- a/gobject/pygtype.c
+++ b/gobject/pygtype.c
@@ -95,7 +95,7 @@ _wrap_g_type_wrapper__get_fundamental(PyGTypeWrapper *self, void *closure)
 static PyObject *
 _wrap_g_type_wrapper__get_children(PyGTypeWrapper *self, void *closure)
 {
-  int n_children, i;
+  guint n_children, i;
   GType *children;
   PyObject *retval;
     
@@ -112,7 +112,7 @@ _wrap_g_type_wrapper__get_children(PyGTypeWrapper *self, void *closure)
 static PyObject *
 _wrap_g_type_wrapper__get_interfaces(PyGTypeWrapper *self, void *closure)
 {
-  int n_interfaces, i;
+  guint n_interfaces, i;
   GType *interfaces;
   PyObject *retval;
     
@@ -774,12 +774,12 @@ pyg_value_from_pyobject(GValue *value, PyObject *obj)
 	}
 	break;
     case G_TYPE_UINT64:
-	g_value_set_uint64(value, PyLong_AsUnsignedLongLong(obj));
-	if (PyErr_Occurred()) {
-	    g_value_unset(value);
-	    PyErr_Clear();
-	    return -1;
-	}
+        if (PyInt_Check(obj))
+            g_value_set_uint64(value, PyInt_AsLong(obj));
+        else if (PyLong_Check(obj))
+            g_value_set_uint64(value, PyLong_AsUnsignedLongLong(obj));
+        else
+            return -1;
 	break;
     case G_TYPE_ENUM:
 	{
@@ -793,7 +793,7 @@ pyg_value_from_pyobject(GValue *value, PyObject *obj)
 	break;
     case G_TYPE_FLAGS:
 	{
-	    guint val = 0;
+	    gint val = 0;
 	    if (pyg_flags_get_value(G_VALUE_TYPE(value), obj, &val) < 0) {
 		PyErr_Clear();
 		return -1;
diff --git a/tests/test_properties.py b/tests/test_properties.py
index 0df6481c..de290bee 100644
--- a/tests/test_properties.py
+++ b/tests/test_properties.py
@@ -44,7 +44,7 @@ class PropertyObject(GObject):
         elif pspec.name == 'construct-only':
             self._construct_only = value
         elif pspec.name == 'uint64':
-            self._uint64
+            self._uint64 = value
         else:
             raise AssertionError
         
@@ -102,10 +102,11 @@ class TestProperties(unittest.TestCase):
         self.assertRaises(TypeError,
                           obj.set_property, 'construct-only', '456')
 
-    def _testUint64(self):
+    def testUint64(self):
         obj = new(PropertyObject)
         self.assertEqual(obj.props.uint64, 0)
         obj.props.uint64 = 1L
         self.assertEqual(obj.props.uint64, 1L)
         obj.props.uint64 = 1
         self.assertEqual(obj.props.uint64, 1L)
+