Hi Marton,

Thanks a lot for your comments. I have modified the patch as per all your 
comments, except the below one.

>> +            tgt = teletext_data_unit_from_ancillary_packet(buf, buf + 1920, 
>> tgt, cctx->teletext_lines, 0);
>
>this should be
>                tgt = teletext_data_unit_from_ancillary_packet(buf, buf + len, 
> tgt, cctx->teletext_lines, 1);
>

I have modified it as

tgt = teletext_data_unit_from_ancillary_packet(buf, buf + len, tgt, 
cctx->teletext_lines, 0);

That was just to be in consistency with the existing code. Because current code 
passes allow_multipacket as 0. Let me know if my understanding here is correct.

Please find the patch attached.

Regards,
Karthick



Attachment: 0003-avdevice-decklink_dec-Added-Closed-caption-decode-fr.patch
Description: 0003-avdevice-decklink_dec-Added-Closed-caption-decode-fr.patch

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to