Hello Evan, what you do is irrelevant you need to define the method signature. See example from Zend/zend_interfaces.c: static ZEND_BEGIN_ARG_INFO(arginfo_arrayaccess_offset, 0) ZEND_ARG_INFO(0, offset) ZEND_END_ARG_INFO();
static ZEND_BEGIN_ARG_INFO(arginfo_arrayaccess_offset_value, 0) ZEND_ARG_INFO(0, offset) ZEND_ARG_INFO(0, value) ZEND_END_ARG_INFO(); zend_function_entry zend_funcs_arrayaccess[] = { ZEND_ABSTRACT_ME(arrayaccess, offsetExists, arginfo_arrayaccess_offset) ZEND_ABSTRACT_ME(arrayaccess, offsetGet, arginfo_arrayaccess_offset) ZEND_ABSTRACT_ME(arrayaccess, offsetSet, arginfo_arrayaccess_offset_value) ZEND_ABSTRACT_ME(arrayaccess, offsetUnset, arginfo_arrayaccess_offset) {NULL, NULL, NULL} }; Tuesday, November 16, 2004, 11:35:24 PM, you wrote: > On Tuesday 16 Nov 2004 13:32, Marcus Boerger wrote: >> Hello Evan, >> >> your implementation is wrong, you need to specify the arguments to >> __get/__set. > Thanks for the reply, but can you be a bit more specific? I'm already > specifying num_args and required_num_args (the former of which is what is > checked in zend_check_magic_method_implementation()). Setting arg_info seems > to have no effect. Here's the code I used to try setting arg_info (apologies > if it wraps) for the __set method: > fe_set.type = ZEND_INTERNAL_FUNCTION; > fe_set.handler = ZEND_FN(fannOO___set); > fe_set.function_name = NULL; > fe_set.scope = NULL; > fe_set.fn_flags = 0; > fe_set.prototype = NULL; > fe_set.num_args = 2; > fe_set.required_num_args = 2; > fe_set.arg_info = ecalloc(fe_set.num_args, sizeof(zend_arg_info)); > fe_set.pass_rest_by_reference = 0; > fe_set.arg_info[0].name = one; > fe_set.arg_info[0].name_len = strlen(one); > fe_set.arg_info[0].class_name = le_fann_name; > fe_set.arg_info[0].class_name_len = strlen(le_fann_name); > fe_set.arg_info[0].allow_null = 0; > fe_set.arg_info[0].pass_by_reference = 0; > fe_set.arg_info[0].return_reference = 0; > fe_set.arg_info[0].required_num_args = fe_set.required_num_args; > fe_set.arg_info[1].name = two; > fe_set.arg_info[1].name_len = strlen(two); > fe_set.arg_info[1].class_name = le_fann_name; > fe_set.arg_info[1].class_name_len = strlen(le_fann_name); > fe_set.arg_info[1].allow_null = 0; > fe_set.arg_info[1].pass_by_reference = 0; > fe_set.arg_info[1].return_reference = 0; > fe_set.arg_info[1].required_num_args = fe_set.required_num_args; >> >> best regards >> marcus >> >> Tuesday, November 16, 2004, 9:01:25 PM, you wrote: >> > Hi everyone, >> > >> > I noticed recently that both of my extensions (Panda and FANN, both in >> > PECL) emit error messages when their __get() and __set() methods are >> > registered. The problem seems to be that fptr.common->num_args is always >> > 0 in zend_check_magic_method_implementation(), regardless of what I set >> > them to prior to registration. >> > >> > The functions still work perfectly, and valgrind doesn't complain. >> > There's just an incredibly annoying error message every time. It's >> > entirely possible (if not probable) that it's my code that's wrong, not >> > ZE2, but I can't for the life of me figure it out. >> > >> > The pertinant portion of the code is based on the SOAP extension. The >> > source code for the fann extension is at >> > >> http://cvs.sourceforge.net/viewcvs.py/*checkout*/fann/fannphp/fann.c?rev= >> >1.13 >> > >> > I tried pecl-dev last week, and the only suggestion I got was using the >> > read_property and write_property callbacks instead of the __get() and >> > __post() methods, but I don't really think that's an option. The thread >> > is at http://marc.theaimsgroup.com/?l=pecl-dev&m=110012046408453&w=2 >> > >> > Thanks. >> > >> > >> > -- >> > Evan Nemerson >> > [EMAIL PROTECTED] >> > http://coeusgroup.com/en >> >> -- >> Best regards, >> Marcus mailto:[EMAIL PROTECTED] > -- > Evan Nemerson > [EMAIL PROTECTED] > http://coeusgroup.com/en -- Best regards, Marcus mailto:[EMAIL PROTECTED] -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: http://www.php.net/unsub.php