Hi Kieran, On Fri, Jun 30, 2023 at 6:41 PM Kieran Kunhya <kier...@obe.tv> wrote: > > On Fri, 30 Jun 2023 at 21:42, Devin Heitmueller > <devin.heitmuel...@ltnglobal.com> wrote: >> >> When encoding to V210, make sure the bar side data makes it through >> in the resulting AVPacket. This is needed so the decklink output >> module can put out bar data as VANC when in 10-bit mode. > > > ff_decode_frame_props_from_pkt surely does this better than the mess of ifs > in v210enc?
The ff_decode_frame_props_from_pkt() function is used when going from AVPackets to AVFrames. The v210enc codec goes in the other direction - from AVFrames to AVPackets. In fact ff_decode_frame_props_from_pkt() does get used by v210dec. Now I wouldn't necessarily be against somebody introducing a new function which goes in the opposite direction and could potentially be shared by other encoders. But my intent was just to implement it the same way as the other conversions were done, not to refactor the code and design some new approach. Devin -- Devin Heitmueller, Senior Software Engineer LTN Global Communications o: +1 (301) 363-1001 w: https://ltnglobal.com e: devin.heitmuel...@ltnglobal.com _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".