<br><br><div class="gmail_quote">On 20 February 2010 18:20, Paul Harrison <span dir="ltr"><<a href="mailto:mythtv@sky.com">mythtv@sky.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<div class="im">Ian Clark wrote:<br>
><br>
><br>
> On 19 February 2010 18:55, Paul Harrison <<a href="mailto:mythtv@sky.com">mythtv@sky.com</a><br>
</div><div><div></div><div class="h5">> <mailto:<a href="mailto:mythtv@sky.com">mythtv@sky.com</a>>> wrote:<br>
><br>
> [snip]<br>
><br>
> OK, thanks for taking the time to produce the log. It does look like a<br>
> problem with the new MythSignalingTimer leaking memory when the main<br>
> thread in mythwelcome isn't running. I think it's because the<br>
> MythSignalingTimer is run in it's own thread and so continues to emit<br>
> signals even when the main thread isn't running and so the signals<br>
> just<br>
> get queued up waiting for the main thread to resume.<br>
><br>
> That's no problem, thanks for debugging it for me.<br>
><br>
><br>
> So the fix looks like it would be to stop the MythSignalingTimer in<br>
> MythMainWindow when mythsystem() is used to start an external process<br>
> and resume it again when it finishes. I'll have a look at trying this<br>
> over the weekend.<br>
><br>
> FWIW, I popped a changeEvent handler on MythMainWindow and did<br>
> 'd->drawTimer->stop()' on activation loss, which seems to have cured<br>
> the problem for mythwelcome (although not for mythfrontend launching<br>
> xbmc, but I suspect that's due to the dirtyness of my patch.) This<br>
> does however solve the main problem of not being able to do anything<br>
> with the remote that I was having. (ssh to reboot all the time is<br>
> tedious.) so I'm happy.<br>
><br>
> Thanks again for the clear and concise fix. :)<br>
><br>
> Cheers,<br>
><br>
> Ian<br>
<br>
<br>
</div></div>I've just committed a fix for this problem to trunk. It should be<br>
generic enough to fix all cases where either MythWelcome or MythFrontend<br>
runs external programs so long as they use myth_system() to start the<br>
process which they do in all the cases that I know of.<br></blockquote><div><br>Cheers for that, I was just atttempting another half assed fix, as my first attempt didn't work as expected, but I shall just svn up now.<br>
<br>Thanks again. :)<br><br>Ian<br><br></div></div>