[MPlayer-dev-eng] [PATCH] libbs2b audio filter
Reimar Döffinger
Reimar.Doeffinger at gmx.de
Mon Mar 23 19:52:41 CET 2009
On Mon, Mar 23, 2009 at 06:27:42PM +0300, Andrew Savchenko wrote:
> IMO the best solution is to replace all occurences of
> malloc/calloc/realloc to their safe wrappers, e.g.
> xmalloc/xcalloc/xrealloc with return value check and fatal error
> message and exit() call if NULL is returned; if sane exit attempt
> is really necessary, signal may be issued and later proceed by
> sighandler. I do not know why this approach isn't used throughout
> all mplayer's code.
What is it supposed to gain? The error message? It's questionable if it
will be possible to print it. I complained about the checks like you did
them because while they are little code, the gains are even less IMHO,
and both no code or a little bit more code to do it right seem like much
more sensible in the effort vs. gain comparison.
More information about the MPlayer-dev-eng
mailing list