Jason Tackaberry wrote: > On Tue, 2005-08-02 at 15:08 +0200, Dirk Meyer wrote: >> But back to the signals. Maybe remove the stdout and stderr functions >> to inherit to and also remove the shutdown callback. There isn't much >> difference between this callback and to connect to the signal. > > True. I didn't want to change the existing API since it might break > other stuff :)
No, don't break it. I will change some freevo code first. >> > add readlines() function to Process class to read lines from process >> > immediately rather than going through the notifier loop; >> >> Why? > [...] >> This is ugly. Maybe it blocks. Is there a good reason why kaa-mplayer >> uses this? > > It does block, that's the point. See kaa.mplayer.MPlayer constructor: I > need to run MPlayer and read some data from it immediately, because if I > don't like what I see, I need to raise an exception saying the MPlayer > executable doesn't look right. I suppose I could use popen directly for > this, if you're really opposed to this addition. But maybe mplayer has some problem starting and it takes 2 seconds for mplayer to start (maybe nfs problems). Blocking here would block freevo. Bad! Dischi -- Time flies... after you hit the snooze button.
pgpGM2mt7bEuY.pgp
Description: PGP signature
