# HG changeset patch
# User romansh
# Date 1141638849 0
# Node ID 1deaaa111fbcf8f85d17a86ed7f34d2e7eaa6eb4
# Parent  312b86bc42021aa3835822885a100344035abefa
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.

diff -r 312b86bc4202 -r 1deaaa111fbc dv.c
--- a/dv.c	Mon Mar 06 08:54:33 2006 +0000
+++ b/dv.c	Mon Mar 06 09:54:09 2006 +0000
@@ -241,16 +241,6 @@
     /* 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;