Re: [FFmpeg-devel] [PATCH] h264_mp4toannexb_bsf: Fix start code size of parameter sets.

2018-03-11 Thread Yusuke Nakamura
2018-02-13 1:45 GMT+09:00 Yusuke Nakamura : > Any parameter set shall have start code of at least 4 byte size. > --- > libavcodec/h264_mp4toannexb_bsf.c| 22 +++--- > tests/ref/fate/h264-bsf-mp4toannexb | 2 +- > tests/ref/fate/h264_mp4toan

Re: [FFmpeg-devel] [PATCH] avpacket: Set dst->side_data_elems to 0 within av_packet_copy_props.

2018-02-13 Thread Yusuke Nakamura
2018-02-14 12:11 GMT+09:00 James Almer : > On 2/13/2018 11:43 PM, Yusuke Nakamura wrote: > > This makes you need not call av_init_packet before av_packet_copy_props > like the following. > > > > AVPacket dst; > > av_packet_copy_props(&dst, &src); &

[FFmpeg-devel] [PATCH] avpacket: Set dst->side_data_elems to 0 within av_packet_copy_props.

2018-02-13 Thread Yusuke Nakamura
This makes you need not call av_init_packet before av_packet_copy_props like the following. AVPacket dst; av_packet_copy_props(&dst, &src); --- libavcodec/avpacket.c | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/avpacket.c b/libavcodec/avpacket.c index 90b8215928..1a9be60e20 100

[FFmpeg-devel] [PATCH] h264_mp4toannexb_bsf: Fix start code size of parameter sets.

2018-02-12 Thread Yusuke Nakamura
Any parameter set shall have start code of at least 4 byte size. --- libavcodec/h264_mp4toannexb_bsf.c| 22 +++--- tests/ref/fate/h264-bsf-mp4toannexb | 2 +- tests/ref/fate/h264_mp4toannexb_ticket2991 | 22 +++--- tests/ref/fate/h264_mp4toan

Re: [FFmpeg-devel] [PATCH 1/3] mpegvideo_parser: implement parsing of the picture structure field

2018-02-11 Thread Yusuke Nakamura
2018-02-11 23:37 GMT+09:00 Jan Ekström : > From: Masaki Tanaka > > Lets one receive the proper field order from pictures coded in > field picture mode, until now forcibly set to BFF. > --- > libavcodec/mpegvideo_parser.c | 16 +++- > 1 file changed, 15 insertions(+), 1 deletion(-) >

Re: [FFmpeg-devel] [PATCH] lavf/mov: ignore edit list with duration equals to 0 for covers art

2017-01-19 Thread Yusuke Nakamura
2017-01-19 18:33 GMT+09:00 Matthieu Bouron : > On Thu, Jan 19, 2017 at 04:55:22AM +0900, Yusuke Nakamura wrote: > > 2017-01-18 19:38 GMT+09:00 Matthieu Bouron : > > > > > Discards edit list with duration equals to 0 for video streams with > only > > > one f

Re: [FFmpeg-devel] [PATCH] lavf/mov: ignore edit list with duration equals to 0 for covers art

2017-01-18 Thread Yusuke Nakamura
2017-01-18 19:38 GMT+09:00 Matthieu Bouron : > Discards edit list with duration equals to 0 for video streams with only > one frame and avoid discarding covers art muxed as a single frame video > stream. > --- > Hello, > > The following patch discards single edit list with duration equals to 0 > f

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Set duration to zero if the duration is UINT_MAX

2016-10-31 Thread Yusuke Nakamura
2016-10-27 4:40 GMT+09:00 Sasi Inguva : > Fixes some MP4F files which have duration in mdhd set to UINT_MAX instead > of zero. > What does this patch fix? > > Signed-off-by: Sasi Inguva > --- > libavformat/mov.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/libavformat/mov.c b

Re: [FFmpeg-devel] [PATCH] flv format support mp3 audio with 48khz

2016-09-27 Thread Yusuke Nakamura
2016-09-28 0:01 GMT+09:00 : > From: frankos2 > > --- > libavformat/flvenc.c |8 > 1 files changed, 8 insertions(+), 0 deletions(-) > > diff --git a/libavformat/flvenc.c b/libavformat/flvenc.c > index 99903f5..296426a 100644 > --- a/libavformat/flvenc.c > +++ b/libavformat/flvenc.c >

Re: [FFmpeg-devel] [PATCH 05/21] libavformat/movenc: mov_write_ftyp_tag: write the major brand a compatible brand

2016-08-23 Thread Yusuke Nakamura
2016-08-23 18:03 GMT+09:00 : > From: Erkki Seppälä > > Signed-off-by: Erkki Seppälä > Signed-off-by: OZOPlayer > --- > libavformat/movenc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/libavformat/movenc.c b/libavformat/movenc.c > index 8c4252d..34bc235 100644 > --- a/libavformat

Re: [FFmpeg-devel] [PATCH 18/21] libavformat/movenc: mov_write_audio_tag writes the proper number of channels, not the hardcoded 2

2016-08-23 Thread Yusuke Nakamura
2016-08-23 18:03 GMT+09:00 : > From: Erkki Seppälä > > Signed-off-by: Erkki Seppälä > Signed-off-by: OZOPlayer > --- > libavformat/movenc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/libavformat/movenc.c b/libavformat/movenc.c > index 94d978b..020d13d 100644 > ---

Re: [FFmpeg-devel] [PATCH] lavf/mov: add support for sidx fragment indexes

2015-10-05 Thread Yusuke Nakamura
2015-10-04 1:57 GMT+09:00 Rodger Combs : > Fixes trac #3842 > --- > libavformat/isom.h | 2 + > libavformat/mov.c | 250 > - > 2 files changed, 213 insertions(+), 39 deletions(-) > > diff --git a/libavformat/isom.h b/libavformat/isom.h > inde

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: Add parsing for DDTS atom for DTS audio

2015-09-30 Thread Yusuke Nakamura
2015-09-30 12:05 GMT+09:00 Michael Niedermayer : > On Tue, Sep 29, 2015 at 02:44:38PM -0700, Shawn Singh wrote: > > The DDTS atom is defined in ETSI TS 102 114, v1.4.1, Annex E. > > This is useful for DTS-HD formats, some of which cannot be > > decoded by dcadec.c or libdcadec. > > > > Signed-off-

Re: [FFmpeg-devel] [PATCH] libavformat/mov.c: Add parsing for DDTS atom for DTS audio

2015-09-29 Thread Yusuke Nakamura
2015-09-30 6:44 GMT+09:00 Shawn Singh : > The DDTS atom is defined in ETSI TS 102 114, v1.4.1, Annex E. > This is useful for DTS-HD formats, some of which cannot be > decoded by dcadec.c or libdcadec. > How useful? DTS audio has scalability, and 'ddts' box indicates only one of properties of the

Re: [FFmpeg-devel] [PATCH] libx264: Add option to force IDR frames

2015-08-18 Thread Yusuke Nakamura
2015-08-19 0:20 GMT+09:00 Carl Eugen Hoyos : > Hendrik Leppkes gmail.com> writes: > > > Whats the other reason for forcing a keyframe, ie. > > why don't we always set IDR? > > I don't know if x264 supports it (and I wanted to ask) > but h264 reference frames can be older than the last > I-frame (

Re: [FFmpeg-devel] [PATCH]lavf/mov: Support alac extradata in wave atom

2015-08-03 Thread Yusuke Nakamura
2015-08-03 21:53 GMT+09:00 wm4 : > On Mon, 3 Aug 2015 10:32:02 + (UTC) > Carl Eugen Hoyos wrote: > > > Paul B Mahol gmail.com> writes: > > > > > > Attached patch fixes ticket #4747 for me, > > > > > > I don't know how to detect that the wave > > > > atom contains no frma / alac atom... > > >

Re: [FFmpeg-devel] [PATCH 2/4] mov: check for positive sample->size

2015-05-24 Thread Yusuke Nakamura
2015-05-24 7:04 GMT+09:00 Andreas Cadhalpun < andreas.cadhal...@googlemail.com>: > Without this the mov demuxer can return a large number of empty packets. > > Signed-off-by: Andreas Cadhalpun > --- > libavformat/mov.c | 5 + > 1 file changed, 5 insertions(+) > > diff --git a/libavformat/mov

Re: [FFmpeg-devel] Question about edit-list+fragment explanation from " [PATCH] mov.c: read fragment start dts from fragmented mp4"

2014-10-26 Thread Yusuke Nakamura
2014-10-26 9:36 GMT+09:00 Bryan Huh : > Sorry I am not posting to the same thread directly, I joined the list after > this discussion finished a few weeks ago. > > I was reading Yusuke's explanation (pasted below) regarding the > relationship between PTS, CTS, and DTS in the context of fragments,

Re: [FFmpeg-devel] [PATCH] mov.c: read fragment start dts from fragmented mp4

2014-10-10 Thread Yusuke Nakamura
2014-10-10 13:38 GMT+09:00 Mika Raento : > On 9 October 2014 23:37, Yusuke Nakamura > wrote: > > 2014-10-10 4:49 GMT+09:00 Michael Niedermayer : > > > >> On Thu, Oct 09, 2014 at 09:44:43PM +0200, Michael Niedermayer wrote: > >> > On Thu, Oct 09, 2014

Re: [FFmpeg-devel] [PATCH] mov.c: read fragment start dts from fragmented mp4

2014-10-09 Thread Yusuke Nakamura
2014-10-10 4:49 GMT+09:00 Michael Niedermayer : > On Thu, Oct 09, 2014 at 09:44:43PM +0200, Michael Niedermayer wrote: > > On Thu, Oct 09, 2014 at 06:57:59PM +0300, Mika Raento wrote: > > > If present, an MFRA box and its TFRAs are read for fragment start > times. > > > > > > Without this change,

Re: [FFmpeg-devel] [PATCH] avformat/movenc: add EAC3 muxing support.

2014-10-02 Thread Yusuke Nakamura
2014-10-02 19:39 GMT+09:00 Benoit Fouet : > Fixes ticket #3074 > --- > libavformat/isom.c | 1 + > libavformat/movenc.c | 44 +++- > 2 files changed, 44 insertions(+), 1 deletion(-) > > diff --git a/libavformat/isom.c b/libavformat/isom.c > index d768c32

Re: [FFmpeg-devel] [PATCH] avformat/movenc: add EAC3 muxing support.

2014-10-02 Thread Yusuke Nakamura
2014-10-02 19:39 GMT+09:00 Benoit Fouet : > Fixes ticket #3074 > --- > libavformat/isom.c | 1 + > libavformat/movenc.c | 44 +++- > 2 files changed, 44 insertions(+), 1 deletion(-) > > diff --git a/libavformat/isom.c b/libavformat/isom.c > index d768c32

[FFmpeg-devel] [PATCH] jpeg2000: Fix a missing compilation of jpeg2000dsp when disabling the encoder.

2014-10-02 Thread Yusuke Nakamura
--- libavcodec/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 04a7944..c6cb6e2 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -277,7 +277,7 @@ OBJS-$(CONFIG_JACOSUB_DECODER) += jacosubdec.o ass.o

Re: [FFmpeg-devel] [PATCH] avformat/movenc: add EAC3 muxing support.

2014-10-02 Thread Yusuke Nakamura
2014-10-02 19:39 GMT+09:00 Benoit Fouet : > Fixes ticket #3074 > --- > libavformat/isom.c | 1 + > libavformat/movenc.c | 44 +++- > 2 files changed, 44 insertions(+), 1 deletion(-) > > diff --git a/libavformat/isom.c b/libavformat/isom.c > index d768c32