From: Marcelo M. <mm...@ac...> - 2005-12-21 18:05:38
|
Sorry, I though I added an entry in the CHANGES.current. I will update it, thanks, Marcelo Olly Betts wrote: >In the CVS HEAD, -nodefault no longer disables wrapping the default >destructor. This was a deliberate change: > >Revision 1.124 - (view) (download) (annotate) - [select for diffs] >Fri Dec 9 09:20:43 2005 UTC (12 days, 7 hours ago) by marcelomatus >Branch: MAIN >CVS Tags: HEAD >Changes since 1.123: +3 -1 lines >Diff to previous 1.123 >add the destructor even when -nodefault is used > >http://cvs.sf.net/viewcvs.py/swig/SWIG/Source/Modules/lang.cxx?r1=1.123&r2=1.124 > >I noticed the change while tracing what looks like a bug in the PHP >backend (more on this once I've dug further) - it's not actually causing >problems for me in real world use. I don't know why the change was made >- it's quite possible it is desirable but it's hard to know without >knowing the motivation behind it. But it does now mean that the >documentation for -nodefault is incorrect. > >Assuming the change in behaviour is indeed desirable, Java.html, >SWIG.html and SWIGPlus.html all need updating, and the change probably >should be listed in CHANGES.current to alert upgrading users. > >Cheers, > Olly > > > >------------------------------------------------------- >This SF.net email is sponsored by: Splunk Inc. Do you grep through log files >for problems? Stop! Download the new AJAX search engine that makes >searching your log files as easy as surfing the web. DOWNLOAD SPLUNK! >http://ads.osdn.com/?ad_id=7637&alloc_id=16865&op=click >_______________________________________________ >Swig-devel mailing list >Swi...@li... >https://lists.sourceforge.net/lists/listinfo/swig-devel > > |