On Tue, Aug 14, 2012 at 11:52:54AM +0100, Måns Rullgård wrote: > Diego Biurrun <[email protected]> writes: > > On Mon, Aug 13, 2012 at 07:40:38PM +0100, Måns Rullgård wrote: > >> Diego Biurrun <[email protected]> writes: > >> > >> > A new hidden config variable is added for the codecs that depend on > >> > the error resilience parts. > >> > --- > >> > configure | 43 ++++++++++++++++++++------------------- > >> > libavcodec/Makefile | 55 > >> > ++++++++++++++++++-------------------------------- > >> > 2 files changed, 42 insertions(+), 56 deletions(-) > >> > >> [...] > >> > >> > +OBJS-$(CONFIG_ERROR_RESILIENCE) += error_resilience.o > >> > >> For a single file, what does this gain? > > > > The same could be said for most of the parts needed by different > > codecs... > > Some of those have a config symbol for other reasons. Anyhow, if you > prefer it this way I don't particularly mind.
We could either develop a strategy now and come up with criteria when to assign a config symbol or we can continue down the current, slightly ad-hoc, path. Right now I'd say we assign config symbols to identifiable self-contained features, so I believe error resilience and the range coder fit the bill. Diego _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
