[FFmpeg-devel] [PATCH v5 1/6] ccfifo: Properly handle CEA-708 captions through framerate conversion

[Devin Heitmueller]

On Fri, May 5, 2023 at 10:10 AM Lance Wang <lance.lmwang at gmail.com> wrote:
> > +int ff_ccfifo_inject(AVCCFifo *ccf, AVFrame *frame)
> > +{
> > +    AVFrameSideData *sd;
> > +    int ret;
> > +
> > +    if (ccf->passthrough == 1 || ccf->cc_detected == 0)
> > +        return 0;
> > +
> > +    sd = av_frame_new_side_data(frame, AV_FRAME_DATA_A53_CC,
> > +                                ff_ccfifo_getoutputsize(ccf));
> > +    if (sd) {
> > +        ret = ff_ccfifo_injectbytes(ccf, sd->data, sd->size);
> > +        if (ret < 0) {
> > +            av_frame_remove_side_data(frame, AV_FRAME_DATA_A53_CC);
> > +            return AVERROR(ENOMEM);
> >
>
> prefer to return ret from ff_ccfifo_injectbytes()

Ok.  I've got to regenerate patch #1 in the series anyway because of
the fate bug Michael reported, so it's easy enough to change this to
"return ret".

Thanks,

Devin

-- 
Devin Heitmueller, Senior Software Engineer
LTN Global Communications
o: +1 (301) 363-1001
w: https://ltnglobal.com  e: devin.heitmueller at ltnglobal.com