Hi, On Wed, Jan 9, 2013 at 5:24 AM, Kieran Kunhya <[email protected]> wrote: >> mpeg-specific crap, most probably useless => remove: >> int8_t *qscale_table; >> int qstride; >> int qscale_type; >> uint8_t *mbskip_table; >> int16_t (*motion_val[2])[2]; >> uint32_t *mb_type; >> short *dct_coeff; >> int8_t *ref_index[2]; >> uint8_t motion_subsample_log2; > > How are things like motion-vectors MPEG specific?
It's fine to save them, but it has no place in the generic AVFrame struct, it should be in a more specialized substruct that is e.g. mpeg-specific. Half of our codecs are intra-only lossless codecs that compress by paletization, thus exposing mpeg-specific crap in a generic struct is kind of ... Lame. Ronald _______________________________________________ libav-devel mailing list [email protected] https://lists.libav.org/mailman/listinfo/libav-devel
