On Mon, Aug 09, 2010 at 01:20:57PM -0400, Ronald S. Bultje wrote: > > + rt->real_setup_cache = av_mallocz(2 * s->nb_streams * sizeof(*rt->real_setup_cache)); > > Why 2 * ..? Because it is also used as a temporary variable to replace a on-stack variable. It might be more readable to add a second pointer to the context instead though.