[FFmpeg-devel] [PATCH] Populate field order returned by avs script, Trac Ticket 8757

emcodem at ffastrans.com emcodem at ffastrans.com
Mon Feb 8 13:45:09 EET 2021


Am 2021-02-01 09:44, schrieb emcodem at ffastrans.com:
> Am 2021-01-21 19:08, schrieb emcodem at ffastrans.com:
>> On 2021-01-21 14:10, Stephen Hutchinson wrote:
>>> Yeah, never mind about that.  I didn't notice that those are declared
>>> AVSC_INLINE, not AVSC_API, so they don't get used through the dynamic
>>> API loader.
>>> 
>>> The comment formatting seems to have been messed up in the second
>>> version, though.
>>> 
>>> /* The following typically only works when assumetff (-bff) and
>>>  * assumefieldbased is used in-script. Additional
>>>  * logic using GetParity() could deliver more accurate results
>>>  * but also decodes a frame which we want to avoid. */
>> 
>> OK, i have to admit formatting comments is in the top 10 of my
>> greatest weaknesses :D
>> Thanks for your patience and also thanks for telling me directly how
>> the formatting is done correctly.
>> New patch with formatted comment attached
> 
> Is it OK to ping this?

Sorry for Pinging again :-(
I know it's not much that the patch does but the resulting functionality 
from a user perspective can be really useful, especially when using 
programatically calculated avs scripts.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Populate-field-order-returned-by-avs-script-Trac-Tic.patch
Type: text/x-diff
Size: 1566 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20210208/30ad40c1/attachment.patch>


More information about the ffmpeg-devel mailing list