[FFmpeg-devel] [PATCH] remove CONFIG_ZLIB checks from lclenc.c
Diego Biurrun
diego
Sun Jun 7 10:34:21 CEST 2009
On Sat, May 30, 2009 at 08:21:04PM +0200, Reimar D?ffinger wrote:
> currently the lclenc.c code is completely unusable without zlib and also
> will not be compiled if it is not available.
> So I propose to remove all those #if CONFIG_ZLIB and just assume it is
> available.
> I guess it was done like this to ease adding a mszh or whatever encoder,
> but this wasn't done the last five years and I have some doubts it would
> be done in the next 5, and even if I think it is questionable to have
> them share a file just because they share some code.
> Any objections?
Go right ahead, this is what we have done in all other such cases.
> Btw. I also think that the lcldec.c code might be better using
> CONFIG_ZLIB_DECODER instead of CONFIG_ZLIB,
Ummm, it already does...
> having code like
> > if (avctx->codec_id == CODEC_ID_ZLIB)
> > print an error...
> when there is no zlib_decoder and thus no decoder registered for
> CODEC_ID_ZLIB seems like useless bloat and would only "protect" against
> severe usage errors.
Agreed.
Diego
More information about the ffmpeg-devel
mailing list