capi: prevent from directly wrapping Go functions

sbinet · sbinet · commit 5167a7b23e1d · 2020-03-26T15:23:02.000+01:00
Fixes #102.
diff --git a/capi.go b/capi.go
@@ -16,11 +16,13 @@ func Py_BuildValue(format string, args ...interface{}) *PyObject {
 // ml_doc	char *	points to the contents of the docstring
 type PyMethodDef struct {
 	Name  string // name of the method
-	Meth  func(self, args *PyObject) *PyObject
+	Meth  PyCFunction
 	Flags MethodDefFlags
 	Doc   string
 }
 
+type PyCFunction C.PyCFunction
+
 type MethodDefFlags int
 
 const (
diff --git a/heap.go b/heap.go
@@ -24,7 +24,7 @@ func cpyMethodDefs(name string, methods []PyMethodDef) *C.PyMethodDef {
 	for i, meth := range methods {
 		cmeth := C.PyMethodDef{
 			ml_name:  C.CString(meth.Name),
-			ml_meth:  (C.PyCFunction)(unsafe.Pointer(&meth.Meth)),
+			ml_meth:  C.PyCFunction(meth.Meth),
 			ml_flags: C.int(meth.Flags),
 			ml_doc:   C.CString(meth.Doc),
 		}
@@ -43,6 +43,7 @@ func Py_InitModule(name string, methods []PyMethodDef) (*PyObject, error) {
 
 	obj := togo(C._gopy_InitModule(c_mname, cmeths))
 	if obj == nil {
+		PyErr_Print()
 		return nil, errors.New("python: internal error; module creation failed.")
 	}
 	return obj, nil
@@ -59,6 +60,7 @@ func Py_InitModule3(name string, methods []PyMethodDef, doc string) (*PyObject,
 
 	obj := togo(C._gopy_InitModule3(cname, cmeths, cdoc))
 	if obj == nil {
+		PyErr_Print()
 		return nil, errors.New("python: internal error; module creation failed.")
 	}
 	return obj, nil
