Hi Kieran, On Fri, Jun 30, 2023 at 6:41 PM Kieran Kunhya <[email protected]> wrote: > > On Fri, 30 Jun 2023 at 21:42, Devin Heitmueller > <[email protected]> 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: [email protected] _______________________________________________ ffmpeg-devel mailing list [email protected] https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email [email protected] with subject "unsubscribe".
