emileclarkb/hammer
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

map SWIG argument correctly

Browse source
This commit is contained in:
Meredith L. Patterson 2013-12-16 00:39:30 +01:00
parent 42b25f4784
commit aae140a420
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
src/bindings/php/hammer.i
View file

@ -40,7 +40,7 @@
// FIXME raise some error
arg1 = NULL;
} else {
res = SWIG_ConvertPtr(*data, &(arg1[i]), SWIGTYPE_p_HParser_, 0 | 0);
res = SWIG_ConvertPtr(*data, &($1[i]), SWIGTYPE_p_HParser_, 0 | 0);
if (!SWIG_IsOK(res)) {
// TODO do we not *have* SWIG_TypeError?
SWIG_exception_fail(res, "that wasn't an HParser");
@ -170,6 +170,7 @@
zval ret;
// in PHP land, the HAction is passed by its name as a string
if (IS_STRING != Z_TYPE_P((zval*)user_data)) {
printf("user_data wasn't a string");
// FIXME throw some error
return NULL;
}
@ -178,6 +179,7 @@
args[0] = hpt_to_php(p->ast);
int ok = call_user_function(EG(function_table), NULL, callable, &ret, 1, args TSRMLS_CC);
if (ok != SUCCESS) {
printf("call_user_function failed");
// FIXME throw some error
return NULL;
}