[alsa-devel] Pull request for alsa-plugins
gmane at colin.guthr.ie
Thu Sep 4 12:23:09 CEST 2008
Takashi Iwai wrote:
> One of my concerns in the current pulse code is the many use of
> assert(). In many cases, assert() is a wrong choice. For example,
> checking the return value of malloc in assert() is definitely wrong.
> It should be always checked, and should return an error instead of
> aborting the program.
> Yes, alsa-lib has also many assert() although I removed already many.
> Some of them may be still wrong ones. Hopefully this can be sorted
> out later.
There has been a rather large thread on this on PA mailing list over the
last couple days :)
I personally got some stuff wrong, but learned from Lennart's responses.
Certainly I think a lot of the cases where this plugin could have
previously hit an assert have now been removed. Not sure about the
malloc asserts() tho'...
Tribalogic Limited [http://www.tribalogic.net/]
Mandriva Linux Contributor [http://www.mandriva.com/]
PulseAudio Hacker [http://www.pulseaudio.org/]
Trac Hacker [http://trac.edgewall.org/]
More information about the Alsa-devel