Andy Wardley wrote:
> I think I've fixed the bug, so it looks like we now have full support
> for tied objects in the XS Stash.

It turns out my "fixes" introduced a memory leak.  How do I know?
Because t/leak.t told me so.  :-)

In my defence, I was doing what perlguts told me to do (increment the
ref count before storing a hash/list item, and decrement it if the 
hv_store() / av_store() method returns NULL).  But whatever perlguts
said, the code was wrong.  

Removing the offending ref count updates made t/leak.t happy, thereby
demonstrating that the XS Stash now does the same thing as the Perl 
version, but faster.  Job done, I can now go out skateboarding.  

Needless to say, I can't stress enough how important it is to have a 
thorough test suite for your software.  Spend all day debugging, or 
go out skating?  Easy choice  :-)

All Hail the Test Suite!

A


_______________________________________________
templates mailing list
[email protected]
http://lists.template-toolkit.org/mailman/listinfo/templates

Reply via email to