I still don't think this should be here. Note: without this code DV encoder

will accept incorrect pix_fmt's and such, but decoder will work much better
with containers not setting the same attributes. It seems like there will
be a generic mechanism for checking such constraints, but if not I can always
resurrect this check for *encoder* only.

Originally committed as revision 5114 to svn://svn.ffmpeg.org/ffmpeg/trunk

libavcodec/dv.c

@@ -241,16 +241,6 @@ static int dvvideo_init(AVCodecContext *avctx)
     /* XXX: do it only for constant case */
     dv_build_unquantize_tables(s, dsp.idct_permutation);
 
-    /* FIXME: I really don't think this should be here */
-    s->sys = dv_codec_profile(avctx);
-    if(!s->sys) {
-        av_log(avctx, AV_LOG_ERROR, "Cannot determine type of DV output stream\n");
-        return -EINVAL;
-    }
-    avctx->pix_fmt = s->sys->pix_fmt;
-    avctx->bit_rate = av_rescale(s->sys->frame_size * 8,
-                                 s->sys->frame_rate,
-                                 s->sys->frame_rate_base);
     avctx->coded_frame = &s->picture;
     s->avctx= avctx;