Class instantiation fixes.

[ruby-edje.git] / src / rb_edje.c

diff --git a/src/rb_edje.c b/src/rb_edje.c
index 8e0f0b6cc0ea219ae659bf1d817a43b46b9d1e93..0e32b83c18c59bd5955eaca4fd58578781018efc 100644 (file)
--- a/src/rb_edje.c
+++ b/src/rb_edje.c
@@ -1,5 +1,5 @@
 /*
- * $Id: rb_edje.c 332 2005-04-28 18:37:16Z tilman $
+ * $Id: rb_edje.c 355 2006-02-10 18:15:13Z tilman $
  *
  * Copyright (C) 2004 Tilman Sauerbeck (tilman at code-monkey de)
  *
@@ -55,31 +55,49 @@ static void c_free (RbEdje *e)
        edje_shutdown ();
 }
 
+static VALUE c_alloc (VALUE klass)
+{
+       RbEdje *e = NULL;
+
+       edje_init ();
+
+       return Data_Make_Struct (klass, RbEdje, c_mark, c_free, e);
+}
+
 /*
  * call-seq:
  *  Edje::Edje.new(evas) => edje
  *
  * Creates an Edje::Edje object.
  */
-static VALUE c_new (VALUE klass, VALUE evas)
+static VALUE c_init (VALUE self, VALUE evas)
 {
-       VALUE self, argv[1];
-       RbEdje *edje = NULL;
-
        CHECK_CLASS (evas, cEvas);
        GET_OBJ (evas, RbEvas, e);
+       GET_OBJ (self, RbEdje, edje);
 
-       edje_init ();
-
-       self = Data_Make_Struct (klass, RbEdje, c_mark, c_free, edje);
+       /* c_new_from_pointer() might already have initialized the pointer */
+       if (!edje->real.real)
+               edje->real.real = edje_object_add (e->real);
 
-       edje->real.real = edje_object_add (e->real);
        edje->parts = Qnil;
        edje->callbacks = Qnil;
        edje->on_text_changed_cb = Qnil;
 
-       argv[0] = evas;
-       rb_obj_call_init (self, 1, argv);
+       rb_call_super (1, &evas);
+
+       return self;
+}
+
+static VALUE c_new_from_pointer (VALUE klass, VALUE evas, VALUE ptr)
+{
+       VALUE self = rb_obj_alloc (klass);
+
+       GET_OBJ (self, RbEdje, edje);
+
+       edje->real.real = (Evas_Object *) ptr;
+
+       rb_obj_call_init (self, 1, &evas);
 
        return self;
 }
@@ -515,11 +533,12 @@ static VALUE c_send_message (VALUE self, VALUE msg)
 
        CHECK_CLASS (msg, cMsg);
 
-       id = NUM2INT (rb_iv_get (msg, "@id"));
+       id = FIX2INT (rb_iv_get (msg, "@id"));
        v = rb_iv_get (msg, "@value");
 
        type = get_msg_type (v);
-       ary = rb_check_array_type (v);
+       if (!NIL_P (ary = rb_check_array_type (v)))
+               len = RARRAY (ary)->len;
 
        switch (type) {
                case EDJE_MESSAGE_NONE:
@@ -649,7 +668,7 @@ static VALUE c_msg_init (int argc, VALUE *argv, VALUE self)
 
        Check_Type (id, T_FIXNUM);
 
-       rb_iv_set (self, "@id", UINT2NUM (id));
+       rb_iv_set (self, "@id", id);
        rb_iv_set (self, "@value", val);
 
        return self;
@@ -659,7 +678,10 @@ void Init_Edje (void)
 {
        cEdje = rb_define_class_under (mEdje, "Edje", cEvasObject);
 
-       rb_define_singleton_method (cEdje, "new", c_new, 1);
+       rb_define_alloc_func (cEdje, c_alloc);
+       rb_define_singleton_method (cEdje, "new_from_pointer",
+                                   c_new_from_pointer, 2);
+       rb_define_method (cEdje, "initialize", c_init, 1);
        rb_define_method (cEdje, "freeze", c_freeze, 0);
        rb_define_method (cEdje, "thaw", c_thaw, 0);
        rb_define_method (cEdje, "load", c_load, 2);