[rt #69243] use the HV * typemap for DSO_call()

tonycoz · May 22, 2013 · 75cebca · 75cebca
1 parent 0aaa8b4
commit 75cebca
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/Changes b/Changes
@@ -23,6 +23,9 @@ Imager release history.  Older releases can be found in Changes.old
    - The XS for i_map() and i_matrix_transform() now use the AV *
      typemap instead of rolling their own.
 
+   - The XS for DSO_call() now uses the HV * typemap instead of
+     rolling it's own.
+
    - The XS for i_poly_aa(), i_poly_aa_cfill(), i_transform() and
      i_gradgen() now use a new T_AVARRAY typemap that greatly
      simplifies the XS code.

diff --git a/Imager.xs b/Imager.xs
@@ -2927,12 +2927,8 @@ void
 DSO_call(handle,func_index,hv)
 	       void*  handle
 	       int    func_index
-	     PREINIT:
-	       HV* hv;
+	       HV *hv
 	     PPCODE:
-	       if (!SvROK(ST(2))) croak("Imager: Parameter 2 must be a reference to a hash\n");	       
-	       hv=(HV*)SvRV(ST(2));
-	       if (SvTYPE(hv)!=SVt_PVHV) croak("Imager: Parameter 2 must be a reference to a hash\n");
 	       DSO_call( (DSO_handle *)handle,func_index,hv);
 
 SV *