Andreas Rottmann | 19 Jul 20:01
Picon
Picon
Gravatar

[Ralf Engels] zinf musicbrowser.cpp


This looks reasonable to me. Thoughts?

Picon Picon
From: Ralf Engels <ralf-engels <at> gmx.de>
Subject: zinf musicbrowser.cpp
Date: 2003-07-18 20:39:10 GMT
    Hi,
I compiled the CVS version from 17-July and got a segmentation fault 
which could be tracked to musicbrowser.cpp

When I started zinf I got the message:
"Parsing the Theme description failed: ..."
The error is probably connected to my old freeamp installation.

However afterwards I got the segmentation fault in 
MuiscBrowserUI::AcceptEvents on the line
delete mainBrowser

I am not sure what exactly the error was because the stack got corrupted.
However the attached patch solves the issue.

Could you please check my correction and check the changes in.
(Continue reading)

Kristian Kvilekval | 20 Jul 02:35
Picon
Gravatar

Re: [Ralf Engels] zinf musicbrowser.cpp

Look fine to me too.. commit it.

On Sat, 2003-07-19 at 11:01, Andreas Rottmann wrote:
> This looks reasonable to me. Thoughts?
> 
> 
> ______________________________________________________________________
> From: Ralf Engels <ralf-engels <at> gmx.de>
> To: Andreas Rottmann <rottmann <at> users.sourceforge.net>
> Subject: zinf musicbrowser.cpp
> Date: 18 Jul 2003 22:39:10 +0200
> 
>     Hi,
> I compiled the CVS version from 17-July and got a segmentation fault 
> which could be tracked to musicbrowser.cpp
> 
> When I started zinf I got the message:
> "Parsing the Theme description failed: ..."
> The error is probably connected to my old freeamp installation.
> 
> However afterwards I got the segmentation fault in 
> MuiscBrowserUI::AcceptEvents on the line
> delete mainBrowser
> 
> I am not sure what exactly the error was because the stack got corrupted.
> However the attached patch solves the issue.
> 
> 
> Could you please check my correction and check the changes in.
> 
(Continue reading)


Gmane