On Thu, 28 Jan 2016 at 16:26 Vittorio Giovara <vittorio.giov...@gmail.com>
wrote:
> On Mon, Jan 25, 2016 at 6:15 PM, Kieran Kunhya <kieran...@googlemail.com>
> wrote:
> >>> +{
> >>> + CFHDContext *s = avctx->priv_data;
> >>> +
> >>> + avctx->pix_fmt = AV_PIX_FMT_YUV422P10;
> >>
> >> if the decoder supports multiple pixel formats it's better not to
> >> initialize the pixel format here, and wait until decode(). Otherwise
> >> it's going to cause a "parameters changed" warning and reinit any
> >> previous filter chain.
> >
> > There are some samples which don't have a pixel format flagged and are
> > implicitly AV_PIX_FMT_YUV422P10.
>
> I still think this should be put off until you get to decoding a
> frame: having a filterchain reinitialization can be disastrous under
> certain conditions.
>
>
Not my problem to handle issues with applications.
> >>> +
> >>> +end:
> >>> + if (ret < 0)
> >>> + return ret;
> >>> +
> >>> + *got_frame = 1;
> >>> + return avpkt->size;
> >>
> >> avpkt->size - bytestream2_get_bytes_left(&gb) no?
> >
> > Guaranteed to read all tags.
>
> if all bytes have been read return 0 then
>
>
eh?
Kieran
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
