Re: Re: music browser

On Fri, 2003-09-19 at 07:01, David Hough wrote:
> Hi,
> 
> In regards to the mention of a memory leak when scrolling any of the
> lists in musicbrowsermm, I don't know if this has been fixed yet(I'm
> having to use a windows only machine at the moment), but looking through
> the code using Browse CVS I think I found where the problem might be.
> 
> In GTKMusicList::scrolledwindow_changed_visible(), the line
> path_returned = m_listMusic.get_path_at_pos(widget_x, widget_y, NewPath,
> TempColumn, widget_x, widget_y);
> fills in the TempColumn pointer. However the object TempColumn points to
> is never deleted, or even used for that matter. So a simple 
> delete TempColumn;
> after that line should fix the problem, hopefully.

Unfortunatley that wasn't,   delete'ing that causes an immediate
crash.   I think it will be somewhere in the creation of
cataloginfo items when the items become visible, we should only
do this the first time.


 
email:[email protected] office:(805)893-4276 http://www.cs.ucsb.edu/~kris



-------------------------------------------------------
This sf.net email is sponsored by:ThinkGeek
Welcome to geek heaven.
http://thinkgeek.com/sf