Re: [FFmpeg-devel] Make disposition forced work with mov_text subtitles.

2022-07-18 Thread facefunk
I see what I did wrong. -v or --reroll-count are arguments to git format-patch so obviously aren't applied by git send-email when providing a preformatted patch, which I did. My mistake. On Mon, 18 Jul 2022 at 18:48, Leo Izen wrote: > On 7/18/22 12:08, facefunk wrote: > > We are not currently ab

[FFmpeg-devel] Make disposition forced work with mov_text subtitles.

2022-07-18 Thread facefunk
Okay, great! Thanks for reviewing. I've addressed your points as well as a few tweaks in the spirit of preserving the abstraction boundary. Would the attached patch revision be acceptable? ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://

Re: [FFmpeg-devel] Make disposition forced work with mov_text subtitles.

2022-07-18 Thread facefunk
Thanks for the advice. I did --reroll-count=2 but it doesn't seem to have worked. On Mon, 18 Jul 2022 at 18:48, Leo Izen wrote: > On 7/18/22 12:08, facefunk wrote: > > We are not currently able to force mov_text subtitles by setting > -disposition:s:0 +forced or equivalent. By setting the forced

Re: [FFmpeg-devel] Make disposition forced work with mov_text subtitles.

2022-07-18 Thread Leo Izen
On 7/18/22 12:08, facefunk wrote: We are not currently able to force mov_text subtitles by setting -disposition:s:0 +forced or equivalent. By setting the forced flags in movtextenc as specifid in https://developer.apple.com/library/archive/documentation/QuickTime/QTFF/QTFFChap3/qtff3.html sub

[FFmpeg-devel] Make disposition forced work with mov_text subtitles.

2022-07-18 Thread facefunk
We are not currently able to force mov_text subtitles by setting -disposition:s:0 +forced or equivalent. By setting the forced flags in movtextenc as specifid in https://developer.apple.com/library/archive/documentation/QuickTime/QTFF/QTFFChap3/qtff3.html subtitles can be forced as expected in