[PD-dev] [ pure-data-Patches-1368465 ] fix memory leak in bonk~

[SourceForge.net]

Patches item #1368465, was opened at 2005-11-28 10:18
Message generated for change (Comment added) made by millerpuckette
You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1368465&group_id=55736

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: puredata
Group: bugfix
Status: Open
Resolution: None
Priority: 5
Submitted By: Mathieu Bouchard (matju)
Assigned to: Miller Puckette (millerpuckette)
Summary: fix memory leak in bonk~

Initial Comment:
fix memory leak in bonk~ 
remove unused variables 
other changes that i dunno about because i diffed 
with MAIN instead of just my own changes. if you want 
to see just my changes, see the pd-cvs archive. 
 

----------------------------------------------------------------------

>Comment By: Miller Puckette (millerpuckette)
Date: 2006-03-23 14:23

Message:
Logged In: YES 
user_id=313747

fixed memory leak.  Haven't fixed unused vars yet.



----------------------------------------------------------------------

Comment By: Thomas Grill (xovo)
Date: 2005-11-30 06:09

Message:
Logged In: YES 
user_id=350252

here's the missing patch (please mind eventual line breaks):

Index: bonk~/bonk~.c
===================================================================
RCS file: /cvsroot/pure-data/pd/extra/bonk~/bonk~.c,v
retrieving revision 1.3.4.4
diff -u -r1.3.4.4 bonk~.c
--- bonk~/bonk~.c	30 Nov 2005 13:57:09 -0000	1.3.4.4
+++ bonk~/bonk~.c	30 Nov 2005 14:06:30 -0000
@@ -950,7 +950,7 @@
 
 void bonk_tilde_setup(void)
 {
-    bonk_class = class_new(gensym("bonk~"),
(t_newmethod)bonk_new, 0,
+    bonk_class = class_new(gensym("bonk~"),
(t_newmethod)bonk_new, (t_method)bonk_free,
         sizeof(t_bonk), 0, A_DEFFLOAT, A_DEFFLOAT, 0);
     class_addmethod(bonk_class, nullfn, gensym("signal"), 0);
     class_addmethod(bonk_class, (t_method)bonk_dsp,
gensym("dsp"), 0);


----------------------------------------------------------------------

You can respond by visiting: 
https://sourceforge.net/tracker/?func=detail&atid=478072&aid=1368465&group_id=55736