On 2009/03/07 19:51, David Guibert <david.guib...@gmail.com> wrote: > I changed the daemonize call as the fork losts the connection to the > pulse server. According to your remark, the init() method should be > moved to the open() ones.
That's better! And hey, the code has become simpler now, because you don't need all the cleanup in the error handlers anymore. By the way, why is close() empty? Don't you need to clean up all the objects? I didn't run valgrind yet to check for memory leaks.. The NULL assignments in pulse_mixer_finish() are pointless, because you're freeing the memory afterwards. Nobody will access it anymore (unless bugged). Max ------------------------------------------------------------------------------ Open Source Business Conference (OSBC), March 24-25, 2009, San Francisco, CA -OSBC tackles the biggest issue in open source: Open Sourcing the Enterprise -Strategies to boost innovation and cut costs with open source participation -Receive a $600 discount off the registration fee with the source code: SFAD http://p.sf.net/sfu/XcvMzF8H _______________________________________________ Musicpd-dev-team mailing list Musicpd-dev-team@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/musicpd-dev-team