Re: [FFmpeg-devel] [PATCH] doc/muxers: add missing MOV muxer options

2020-03-14 Thread Gyan Doshi
On 15-03-2020 12:47 am, Lou Logan wrote: On Fri, Mar 13, 2020, at 10:23 PM, Gyan Doshi wrote: Do you mind if I continue with my own patch? I started reviewing this and noticed that many of the option description should have caveats since they can be silently (i.e. unlogged) modified based on o

Re: [FFmpeg-devel] [PATCH] doc/muxers: add missing MOV muxer options

2020-03-14 Thread Lou Logan
On Fri, Mar 13, 2020, at 10:23 PM, Gyan Doshi wrote: > > Do you mind if I continue with my own patch? I started reviewing this > and noticed that many of the option description should have caveats > since they can be silently (i.e. unlogged) modified based on other > options. I think it's best f

Re: [FFmpeg-devel] [PATCH] doc/muxers: add missing MOV muxer options

2020-03-13 Thread Gyan Doshi
On 13-03-2020 10:02 am, Gyan Doshi wrote: On 13-03-2020 12:08 am, Lou Logan wrote: Signed-off-by: Lou Logan --- The options for this muxer have a separate section about fragmenting. I tried to organize the missing options into the proper sections, but I don't use fragmenting so I may have p

Re: [FFmpeg-devel] [PATCH] doc/muxers: add missing MOV muxer options

2020-03-13 Thread Lou Logan
On Thu, Mar 12, 2020, at 8:32 PM, Gyan Doshi wrote: > > I already started on this after I added the MOV demuxer options. > > But this is a good start. I'll probably want to expand descriptions > later on. > > Will have a look and push this weekend. Thanks. Here is my muxer.texi todo list if yo

Re: [FFmpeg-devel] [PATCH] doc/muxers: add missing MOV muxer options

2020-03-12 Thread Gyan Doshi
On 13-03-2020 12:08 am, Lou Logan wrote: Signed-off-by: Lou Logan --- The options for this muxer have a separate section about fragmenting. I tried to organize the missing options into the proper sections, but I don't use fragmenting so I may have placed them in the wrong section. I already

Re: [FFmpeg-devel] [PATCH] doc/muxers: add missing MOV muxer options

2020-03-12 Thread Michael Niedermayer
On Thu, Mar 12, 2020 at 10:38:37AM -0800, Lou Logan wrote: > Signed-off-by: Lou Logan > --- > The options for this muxer have a separate section about fragmenting. > I tried to organize the missing options into the proper sections, but > I don't use fragmenting so I may have placed them in the wro

[FFmpeg-devel] [PATCH] doc/muxers: add missing MOV muxer options

2020-03-12 Thread Lou Logan
Signed-off-by: Lou Logan --- The options for this muxer have a separate section about fragmenting. I tried to organize the missing options into the proper sections, but I don't use fragmenting so I may have placed them in the wrong section. --- doc/muxers.texi | 116 ++