From: William S F. <ws...@fu...> - 2007-03-23 22:12:40
|
Josh Cherry wrote: > > > On Thu, 15 Mar 2007, William S Fulton wrote: > >>> I changed CopyString so that the new string's capacity is only as >>> large as >>> its length. The result was a 25% decrease in SWIG memory >>> consumption, at >>> no significant cost in speed (it looks like it runs very slightly >>> faster, >>> perhaps due to elimination of the unnecessary copying mentioned above). >>> >> Sounds good to me. Running the entire test-suite for all languages you >> possibly can would be a good idea for this kind of modification though. > > I ran the tests for Python, Perl, ruby, and tcl with 1.3.31 and my > change. The results were the same as for unmodified 1.3.31 (I see a Perl > failure and two tcl failures even with the unmodified SWIG). > > Here is my modified CopyString: > I've committed this. Thanks a lot! William |