Implement XS accessor generators

[gitmo/Mouse.git] / xs-src / mouse_simple_accessor.xs

diff --git a/xs-src/mouse_simple_accessor.xs b/xs-src/mouse_simple_accessor.xs
index cf835fb..285f3cb 100644 (file)
--- a/xs-src/mouse_simple_accessor.xs
+++ b/xs-src/mouse_simple_accessor.xs
@@ -1,14 +1,14 @@
-#include "mouse.h"\r
-\r
+#include "mouse.h"
+
 static MGVTBL mouse_simple_accessor_vtbl;
-\r\r
+
 /*
 static MAGIC*
 mouse_accessor_get_mg(pTHX_ CV* const xsub){
     return moose_mg_find(aTHX_ (SV*)xsub, &mouse_simple_accessor_vtbl, MOOSEf_DIE_ON_FAIL);
-}\r
-*/\r
-\r
+}
+*/
+
 CV*
 mouse_install_simple_accessor(pTHX_ const char* const fq_name, const char* const key, I32 const keylen, XSUBADDR_t const accessor_impl){
     CV* const xsub = newXS((char*)fq_name, accessor_impl, __FILE__);
@@ -24,13 +24,13 @@ mouse_install_simple_accessor(pTHX_ const char* const fq_name, const char* const
     SvREFCNT_dec(slot); /* sv_magicext() increases refcnt in mg_obj */
 
     /* NOTE:
-     * although we use MAGIC for gc, we also store slot to CvXSUBANY slot for efficiency (gfx)
+     * although we use MAGIC for gc, we also store mg to CvXSUBANY for efficiency (gfx)
      */
-    CvXSUBANY(xsub).any_ptr = (void*)slot;
+    CvXSUBANY(xsub).any_ptr = (void*)mg;
 
     return xsub;
 }
-\r
+
 SV*
 mouse_accessor_get_self(pTHX_ I32 const ax, I32 const items, CV* const cv) {
     SV* self;
@@ -48,18 +48,18 @@ mouse_accessor_get_self(pTHX_ I32 const ax, I32 const items, CV* const cv) {
         croak("Cant call %s as a class method", GvNAME(CvGV(cv)));
     }
     return self;
-}\r
+}
 
 
 XS(mouse_xs_simple_reader)
 {
-    dVAR; dXSARGS;\r
+    dVAR; dXSARGS;
     dMOUSE_self;
-    SV* const slot = (SV*)XSANY.any_ptr;
+    SV* const slot = MOUSE_mg_slot((MAGIC*)XSANY.any_ptr);
     SV* value;
 
     if (items != 1) {
-        croak("Expected exactly one argument");
+        croak("Expected exactly one argument for a reader for '%"SVf"'", slot);
     }
 
     value = mouse_instance_get_slot(self, slot);
@@ -72,27 +72,42 @@ XS(mouse_xs_simple_writer)
 {
     dVAR; dXSARGS;
     dMOUSE_self;
-    SV* const slot = (SV*)XSANY.any_ptr;
-\r
+    SV* const slot = MOUSE_mg_slot((MAGIC*)XSANY.any_ptr);
+
     if (items != 2) {
-        croak("Expected exactly two argument");
+        croak("Expected exactly two argument for a writer for '%"SVf"'", slot);
     }
 
     ST(0) = mouse_instance_set_slot(self, slot, ST(1));
     XSRETURN(1);
 }
 
+XS(mouse_xs_simple_clearer)
+{
+    dVAR; dXSARGS;
+    dMOUSE_self;
+    SV* const slot = MOUSE_mg_slot((MAGIC*)XSANY.any_ptr);
+    SV* value;
+
+    if (items != 1) {
+        croak("Expected exactly one argument for a clearer for '%"SVf"'", slot);
+    }
+
+    value = mouse_instance_delete_slot(aTHX_ self, slot);
+    ST(0) = value ? value : &PL_sv_undef;
+    XSRETURN(1);
+}
 
 XS(mouse_xs_simple_predicate)
 {
     dVAR; dXSARGS;
     dMOUSE_self;
-    SV* const slot = (SV*)XSANY.any_ptr;
-\r
+    SV* const slot = MOUSE_mg_slot((MAGIC*)XSANY.any_ptr);
+
     if (items != 1) {
-        croak("Expected exactly one argument");
+        croak("Expected exactly one argument for a predicate for '%"SVf"'", slot);
     }
 
     ST(0) = boolSV( mouse_instance_has_slot(self, slot) );
     XSRETURN(1);
-}\r
+}