From: SourceForge.net <no...@so...> - 2012-01-24 22:06:35
|
Bugs item #3479190, was opened at 2012-01-24 13:13 Message generated for change (Comment added) made by olly You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=101645&aid=3479190&group_id=1645 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: php Group: None Status: Closed >Resolution: Fixed Priority: 5 Private: No Submitted By: Boruch Baum (boruch-baum) Assigned to: Olly Betts (olly) Summary: invalid type conversion in foobar_wrap.cpp Initial Comment: Following is output from an attempt to create a binding to php5.4 (5.4.0~rc6-1) using swig v2.0.4-4 and also using the swig svn version 12901. This was done in debian testing (wheezy) with the php5 package installed from debian experimental. See the patch proposed at: http://www.graphviz.org/mantisbt/view.php?id=2177 make[2]: Entering directory `.../libhdate/libhdate-now/trunk/libhdate/bindings/php' swig -php5 -c++ hdate.i g++ -I../../src `php-config --includes` -fpic -c hdate_wrap.cpp hdate_wrap.cpp: In function ‘void* SWIG_ZTS_ConvertResourcePtr(zval*, swig_type_info*, int)’: hdate_wrap.cpp:933:65: error: invalid conversion from ‘const char*’ to ‘char*’ [-fpermissive] make[2]: *** [hdate.so] Error 1 make[2]: Leaving directory `...//libhdate/libhdate-now/trunk/libhdate/bindings/php' make[1]: *** [all-recursive] Error 1 make[1]: Leaving directory `...//libhdate/libhdate-now/trunk/libhdate/bindings' make: *** [all-recursive] Error 1 ---------------------------------------------------------------------- >Comment By: Olly Betts (olly) Date: 2012-01-24 14:06 Message: As a comment on the graphviz ticket notes, this is already fixed in SWIG SVN trunk - the revision for it was 12724 (which was shortly after the 2.0.4 release): $ svn diff -c12724 Index: Lib/php/phprun.swg =================================================================== --- Lib/php/phprun.swg (revision 12723) +++ Lib/php/phprun.swg (revision 12724) @@ -142,7 +142,7 @@ result = zend_lookup_class(classname, SWIG_PREFIX_LEN + type_name_len, &ce TSRMLS_CC); efree(classname); } else { - result = zend_lookup_class(type_name, type_name_len, &ce TSRMLS_CC); + result = zend_lookup_class((char *)type_name, type_name_len, &ce TSRMLS_CC); } if (result != SUCCESS) { /* class does not exist */ ---------------------------------------------------------------------- Comment By: Olly Betts (olly) Date: 2012-01-24 14:06 Message: As a comment on the graphviz ticket notes, this is already fixed in SWIG SVN trunk - the revision for it was 12724 (which was shortly after the 2.0.4 release): $ svn diff -c12724 Index: Lib/php/phprun.swg =================================================================== --- Lib/php/phprun.swg (revision 12723) +++ Lib/php/phprun.swg (revision 12724) @@ -142,7 +142,7 @@ result = zend_lookup_class(classname, SWIG_PREFIX_LEN + type_name_len, &ce TSRMLS_CC); efree(classname); } else { - result = zend_lookup_class(type_name, type_name_len, &ce TSRMLS_CC); + result = zend_lookup_class((char *)type_name, type_name_len, &ce TSRMLS_CC); } if (result != SUCCESS) { /* class does not exist */ ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=101645&aid=3479190&group_id=1645 |