Re: [FFmpeg-devel] [PATCH 3/3] avformat/oggenc: add vp8 muxing support

2016-07-20 Thread James Almer
On 7/12/2016 6:36 PM, James Almer wrote: > Addresses ticket #5687 > > Signed-off-by: James Almer > --- > AVFMT_TS_NONSTRICT is added to flags because invisible vp8 frames don't > increase the pts > > libavformat/oggenc.c| 94 > + > tests/fate

Re: [FFmpeg-devel] [PATCH 3/3] avformat/oggenc: add vp8 muxing support

2016-07-12 Thread James Almer
On 7/12/2016 7:50 PM, Moritz Barsnick wrote: > On Tue, Jul 12, 2016 at 18:36:22 -0300, James Almer wrote: >> +/** for VP8 granule */ > > Spurious asterisk. ;-) Copy paste from a Theora comment a couple lines above this one. Is it a problem? I can change both to use a single asterisk if needed

Re: [FFmpeg-devel] [PATCH 3/3] avformat/oggenc: add vp8 muxing support

2016-07-12 Thread Moritz Barsnick
On Tue, Jul 12, 2016 at 18:36:22 -0300, James Almer wrote: > +/** for VP8 granule */ Spurious asterisk. ;-) ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

[FFmpeg-devel] [PATCH 3/3] avformat/oggenc: add vp8 muxing support

2016-07-12 Thread James Almer
Addresses ticket #5687 Signed-off-by: James Almer --- AVFMT_TS_NONSTRICT is added to flags because invisible vp8 frames don't increase the pts libavformat/oggenc.c| 94 + tests/fate/avformat.mak | 1 + tests/lavf-regression.sh| 4 ++