Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Tobias Rapp
On 28.03.2017 10:19, Clément Bœsch wrote: On Tue, Mar 28, 2017 at 12:47:39AM +0200, Alexander Strasser wrote: [...] + +#define av_4cc2str(fourcc) av_fourcc_make_string((char[AV_FOURCC_MAX_STRING_SIZE]){0}, fourcc) Did your write it as 4cc2str to make the name shorter? I guess fourcc2str is mo

[FFmpeg-devel] [PATCH 1/3] avutil/spherical: add av_spherical_projection_name()

2017-03-28 Thread James Almer
Signed-off-by: James Almer --- doc/APIchanges| 3 +++ libavutil/spherical.c | 15 +++ libavutil/spherical.h | 9 + 3 files changed, 27 insertions(+) diff --git a/doc/APIchanges b/doc/APIchanges index 2274543024..5f3c268d05 100644 --- a/doc/APIchanges +++ b/doc/APIch

[FFmpeg-devel] [PATCH 2/3] avformat/dump: use av_spherical_projection_name() to print spherical projection names

2017-03-28 Thread James Almer
Signed-off-by: James Almer --- libavformat/dump.c | 15 ++- 1 file changed, 6 insertions(+), 9 deletions(-) diff --git a/libavformat/dump.c b/libavformat/dump.c index 3e6218303d..cb0468559e 100644 --- a/libavformat/dump.c +++ b/libavformat/dump.c @@ -353,21 +353,18 @@ static void dum

[FFmpeg-devel] [PATCH 3/3] ffprobe: use av_spherical_projection_name() to print spherical projection names

2017-03-28 Thread James Almer
Signed-off-by: James Almer --- ffprobe.c | 10 +++--- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/ffprobe.c b/ffprobe.c index 67c4ed40fa..be5583aea2 100644 --- a/ffprobe.c +++ b/ffprobe.c @@ -1872,22 +1872,18 @@ static void print_pkt_side_data(WriterContext *w,

Re: [FFmpeg-devel] [PATCH 2/2] avcodec/ac3dsp: add special-case handling for the C downmix_fixed function

2017-03-28 Thread James Almer
On 3/22/2017 10:56 PM, Michael Niedermayer wrote: > On Wed, Mar 22, 2017 at 12:41:26PM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> libavcodec/ac3dsp.c | 61 >> + >> 1 file changed, 61 insertions(+) > > LGTM > > thx Push

Re: [FFmpeg-devel] [PATCH 1/2] fate: add tests for ac3_fixed 5.1 downmix

2017-03-28 Thread James Almer
On 3/22/2017 10:50 PM, Michael Niedermayer wrote: > On Wed, Mar 22, 2017 at 12:41:25PM -0300, James Almer wrote: >> Signed-off-by: James Almer >> --- >> tests/fate/ac3.mak | 8 >> 1 file changed, 8 insertions(+) > > LGTM > > thx Pushed. ___

[FFmpeg-devel] [PATCH] hevc: only write to max_ra and pocTid0 in the first slice.

2017-03-28 Thread Ronald S. Bultje
Values from subsequent values are guaranteed to be identical (since poc and nal_unit_type are checked to be the same between slices), so this doesn't affect output in any way, but does resolve the remaining reported race conditions (by tsan) in fate-hevc. --- libavcodec/hevcdec.c | 32

[FFmpeg-devel] [PATCH] lagarith: assign correct per-thread value to LagarithContext::avctx.

2017-03-28 Thread Ronald S. Bultje
This fixes race conditions reported by tsan in fate-lagarith. The races were because each thread's LagarithContext::avctx was set to the first thread's AVCodecContext. --- libavcodec/lagarith.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/libavcodec/lagarith.c b/libavcodec/laga

[FFmpeg-devel] [PATCH] fic: set pict_type/key_frame after (instead of during) slice decoding.

2017-03-28 Thread Ronald S. Bultje
This fixes a race condition that was already documented in the source code, and is also reported by tsan in fate-fic-avi. --- libavcodec/fic.c | 29 ++--- 1 file changed, 14 insertions(+), 15 deletions(-) diff --git a/libavcodec/fic.c b/libavcodec/fic.c index d3952a4..2bec

[FFmpeg-devel] GIF encoder transparency support

2017-03-28 Thread Jonathan Meson
Hey Guys, I've been working with GIF encoding and was trying to encode GIFs with transparency support but I'm having no luck. I'm getting black instead of transparency. I'm currently using "palettegen" and "paletteuse" filters to create nicer quality GIFs. I have tried using GIF codec flag "giffl

[FFmpeg-devel] [PATCH 1/2] png: split header state and data state in two separate variables.

2017-03-28 Thread Ronald S. Bultje
Fixes a reported (but false) race condition in tsan for fate-apng. --- libavcodec/pngdec.c | 39 +++ 1 file changed, 19 insertions(+), 20 deletions(-) diff --git a/libavcodec/pngdec.c b/libavcodec/pngdec.c index a4eb6cc..bbb9610 100644 --- a/libavcodec/pngdec.c

[FFmpeg-devel] [PATCH] codec_desc: mark fraps as an intra-only codec.

2017-03-28 Thread Ronald S. Bultje
Fixes reported race conditions by tsan in fate-avio-direct. --- libavcodec/codec_desc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavcodec/codec_desc.c b/libavcodec/codec_desc.c index 4e98cf9..9711019 100644 --- a/libavcodec/codec_desc.c +++ b/libavcodec/codec_desc.c @@

[FFmpeg-devel] [PATCH 2/2] dnxhd: initialize DNXHDContext::avctx to each thread's respective one.

2017-03-28 Thread Ronald S. Bultje
Otherwise all thread's private contexts have the avctx pointer set to the AVCodecContext of the first thread, which means all writes to ctx->avctx->* (in e.g. read_header) are effectively race conditions. Fixes fate-dnxhd under tsan. --- libavcodec/dnxhddec.c | 1 + 1 file changed, 1 insertion(+)

Re: [FFmpeg-devel] mpeg TS encoder PCR algo + looping

2017-03-28 Thread Moritz Barsnick
On Tue, Mar 28, 2017 at 18:30:30 +0300, ffm...@scil.sinp.msu.ru wrote: In addition to what Michael wrote: > Attached patch changes the PCR pid dedication number algo: [...] > Also, the title, service name and provider name were corrected in the > ffmpeg_opt.c: options required in mpegtsenc.c wer

[FFmpeg-devel] [PATCH] Add vda_vt_internal.h to SKIPHEADERS.

2017-03-28 Thread Ronald S. Bultje
--- libavcodec/Makefile | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 876a69e..9c14cfa 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -1001,6 +1001,7 @@ SKIPHEADERS+= %_tablegen.h

[FFmpeg-devel] [PATCH]lavf/isom: Remove an incorrect codec point for WMAv2

2017-03-28 Thread Carl Eugen Hoyos
Hi! libavformat contains a codec point for WMAv2 for many years, it allows to write undecodable files but does not allow to decode existing (flip4mac) files. I believe the flip4mac files should be read differently, without using the code point. Please comment, Carl Eugen From 05f8ac8e0d583eeb2

Re: [FFmpeg-devel] [PATCH] lavfi: fix race when func rets holder is NULL

2017-03-28 Thread Clément Bœsch
On Tue, Mar 28, 2017 at 11:39:49PM +0200, Michael Niedermayer wrote: > On Tue, Mar 28, 2017 at 05:59:29PM +0200, Clément Bœsch wrote: > > From: Clément Bœsch > > > > If ret is NULL, a dummy common holder is created to hold *all* the > > parallel function returns, which gets written concurrently.

Re: [FFmpeg-devel] [PATCH] vda: add necessary includes to pass make checkheaders.

2017-03-28 Thread Ronald S. Bultje
Hi, On Tue, Mar 28, 2017 at 5:38 PM, Clément Bœsch wrote: > On Tue, Mar 28, 2017 at 05:35:08PM -0400, Ronald S. Bultje wrote: > > --- > > libavcodec/vda_vt_internal.h | 7 +++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/libavcodec/vda_vt_internal.h b/libavcodec/vda_vt_internal.

Re: [FFmpeg-devel] [PATCH] Add vda_vt_internal.h to SKIPHEADERS.

2017-03-28 Thread Clément Bœsch
On Tue, Mar 28, 2017 at 05:40:50PM -0400, Ronald S. Bultje wrote: > This fixes make checkheaders. > --- > libavcodec/Makefile | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/libavcodec/Makefile b/libavcodec/Makefile > index 876a69e..023fd15 100644 > --- a/libavcodec/Makefile > +++ b/liba

[FFmpeg-devel] [PATCH] Add vda_vt_internal.h to SKIPHEADERS.

2017-03-28 Thread Ronald S. Bultje
This fixes make checkheaders. --- libavcodec/Makefile | 1 + 1 file changed, 1 insertion(+) diff --git a/libavcodec/Makefile b/libavcodec/Makefile index 876a69e..023fd15 100644 --- a/libavcodec/Makefile +++ b/libavcodec/Makefile @@ -1001,6 +1001,7 @@ SKIPHEADERS+= %_ta

Re: [FFmpeg-devel] [PATCH] lavfi: fix race when func rets holder is NULL

2017-03-28 Thread Michael Niedermayer
On Tue, Mar 28, 2017 at 05:59:29PM +0200, Clément Bœsch wrote: > From: Clément Bœsch > > If ret is NULL, a dummy common holder is created to hold *all* the > parallel function returns, which gets written concurrently. This commit > simplify the whole logic by simply not writing to that holder whe

Re: [FFmpeg-devel] [PATCH] vda: add necessary includes to pass make checkheaders.

2017-03-28 Thread Clément Bœsch
On Tue, Mar 28, 2017 at 05:35:08PM -0400, Ronald S. Bultje wrote: > --- > libavcodec/vda_vt_internal.h | 7 +++ > 1 file changed, 7 insertions(+) > > diff --git a/libavcodec/vda_vt_internal.h b/libavcodec/vda_vt_internal.h > index 9ff63cc..458ebb4 100644 > --- a/libavcodec/vda_vt_internal.h >

[FFmpeg-devel] [PATCH] vda: add necessary includes to pass make checkheaders.

2017-03-28 Thread Ronald S. Bultje
--- libavcodec/vda_vt_internal.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/libavcodec/vda_vt_internal.h b/libavcodec/vda_vt_internal.h index 9ff63cc..458ebb4 100644 --- a/libavcodec/vda_vt_internal.h +++ b/libavcodec/vda_vt_internal.h @@ -19,6 +19,13 @@ #ifndef AVCODEC_VDA_VT_INT

[FFmpeg-devel] [PATCH] Added HEVC and Opus support to nut

2017-03-28 Thread Leo Izen
--- libavformat/nut.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavformat/nut.c b/libavformat/nut.c index 04776e249f..592fe4dc28 100644 --- a/libavformat/nut.c +++ b/libavformat/nut.c @@ -42,6 +42,7 @@ const AVCodecTag ff_nut_video_tags[] = { { AV_CODEC_ID_GIF, MKTA

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Matthias Hunstock
Am 28.03.2017 um 12:59 schrieb Clément Bœsch: > I didn't rename the prototype or the macro so it doesn't need any update, > or am I missing something? No you don't.. I did - sorry! Matthias ___ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ff

[FFmpeg-devel] [PATCH] new API usage example (adts-aac encoding from raw audio file)

2017-03-28 Thread Paolo Prete
--- doc/examples/Makefile | 1 + doc/examples/encode_raw_audio_file_to_aac.c | 275 2 files changed, 276 insertions(+) create mode 100644 doc/examples/encode_raw_audio_file_to_aac.c diff --git a/doc/examples/Makefile b/doc/examples/Makefile in

Re: [FFmpeg-devel] [PATCH 08/10] lavf/ape: remove unused magic field

2017-03-28 Thread Michael Niedermayer
On Mon, Mar 27, 2017 at 09:52:01AM +0200, Clément Bœsch wrote: > --- > libavformat/ape.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/libavformat/ape.c b/libavformat/ape.c > index 50a1aa13c1..c06db78480 100644 > --- a/libavformat/ape.c > +++ b/libavformat/ape.c > @@ -57,7 +57,6 @@ ty

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Alexander Strasser
On 2017-03-28 18:16 +0200, Clément Bœsch wrote: > On Tue, Mar 28, 2017 at 06:12:14PM +0200, Michael Niedermayer wrote: > > On Tue, Mar 28, 2017 at 10:19:46AM +0200, Clément Bœsch wrote: > > [...] > > > diff --git a/libavutil/utils.c b/libavutil/utils.c > > > index 36e4dd5fdb..29f2746338 100644 > >

Re: [FFmpeg-devel] [PATCH 1/3] configure: use c++11 and fallback to c++0x for c++ files

2017-03-28 Thread Marton Balint
On Sun, 26 Mar 2017, James Almer wrote: On 3/26/2017 7:22 AM, Marton Balint wrote: On Sat, 25 Mar 2017, James Almer wrote: On 3/25/2017 8:51 PM, Marton Balint wrote: Needed for the C+11 atomics. Also change add_cxxflags to check_cxxflags. Signed-off-by: Marton Balint --- configure | 2 +

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Alexander Strasser
On 2017-03-28 10:19 +0200, Clément Bœsch wrote: > On Tue, Mar 28, 2017 at 12:47:39AM +0200, Alexander Strasser wrote: > [...] > > > +#define AV_FOURCC_MAX_STRING_SIZE 32 > > > > Should be fine, though I don't know how you came up with this max size in > > particular. > > > > IIRC the actual max

Re: [FFmpeg-devel] mpeg TS encoder PCR algo + looping

2017-03-28 Thread Michael Niedermayer
On Tue, Mar 28, 2017 at 06:30:30PM +0300, ffm...@scil.sinp.msu.ru wrote: > > Hi! > > Attached patch changes the PCR pid dedication number algo: > pcr_pid belongs to the program, not to entire TS (it was the mistake > in TS code). So, the patch makes it possible to broadcast TS with > several prog

Re: [FFmpeg-devel] [PATCH] doc/examples/remuxing: switch to codecpar

2017-03-28 Thread Michael Niedermayer
On Tue, Mar 28, 2017 at 04:54:38PM +0200, Matthieu Bouron wrote: > On Tue, Mar 28, 2017 at 12:32 PM, Matthieu Bouron > wrote: > > > Also limits remuxing to audio, video and subtitle streams. > > --- > > doc/examples/remuxing.c | 48 ++ > > +++--- > > 1 fil

[FFmpeg-devel] [PATCH] pthread_frame: don't sync items between threads for intra-only codecs.

2017-03-28 Thread Ronald S. Bultje
Intra-only codecs should either be able to read these items from the bitstream, or they should be set upon codec initialization. In both cases, syncing these items at runtime is unnecessary. In practice, this fixes race conditions for decoders that read these values from the bitstream. --- libavc

[FFmpeg-devel] mpeg TS encoder PCR algo + looping

2017-03-28 Thread ffmpeg
Hi! Attached patch changes the PCR pid dedication number algo: pcr_pid belongs to the program, not to entire TS (it was the mistake in TS code). So, the patch makes it possible to broadcast TS with several programs inside. It is possible to set the pcr pid manually by adding in the prog conf

[FFmpeg-devel] [PATCH] pthread_frame: don't sync items between threads for intra-only codecs.

2017-03-28 Thread Ronald S. Bultje
Intra-only codecs should either be able to read these items from the bitstream, or they should be set upon codec initialization. In both cases, syncing these items at runtime is unnecessary. In practice, this fixes race conditions for decoders that read these values from the bitstream. --- libavc

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Clément Bœsch
On Tue, Mar 28, 2017 at 06:12:14PM +0200, Michael Niedermayer wrote: > On Tue, Mar 28, 2017 at 10:19:46AM +0200, Clément Bœsch wrote: > [...] > > diff --git a/libavutil/utils.c b/libavutil/utils.c > > index 36e4dd5fdb..29f2746338 100644 > > --- a/libavutil/utils.c > > +++ b/libavutil/utils.c > > @@

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Michael Niedermayer
On Tue, Mar 28, 2017 at 10:19:46AM +0200, Clément Bœsch wrote: [...] > diff --git a/libavutil/utils.c b/libavutil/utils.c > index 36e4dd5fdb..29f2746338 100644 > --- a/libavutil/utils.c > +++ b/libavutil/utils.c > @@ -121,6 +121,29 @@ unsigned av_int_list_length_for_size(unsigned elsize, > ret

[FFmpeg-devel] [PATCH] lavfi: fix race when func rets holder is NULL

2017-03-28 Thread Clément Bœsch
From: Clément Bœsch If ret is NULL, a dummy common holder is created to hold *all* the parallel function returns, which gets written concurrently. This commit simplify the whole logic by simply not writing to that holder when not set. --- libavfilter/pthread.c | 17 ++--- 1 file chan

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Clément Bœsch
On Mon, Mar 27, 2017 at 09:51:54AM +0200, Clément Bœsch wrote: > --- > doc/APIchanges | 4 > libavutil/avutil.h | 14 ++ > libavutil/utils.c | 21 + > libavutil/version.h | 2 +- > 4 files changed, 40 insertions(+), 1 deletion(-) > I'll start pushin

Re: [FFmpeg-devel] [PATCH] dirac: make initialization of arithmetic coder tables threadsafe.

2017-03-28 Thread Ronald S. Bultje
Hi, On Mon, Mar 27, 2017 at 11:24 AM, Ronald S. Bultje wrote: > Hi, > > On Mon, Mar 27, 2017 at 11:10 AM, Michael Niedermayer < > mich...@niedermayer.cc> wrote: > >> On Mon, Mar 27, 2017 at 10:09:05AM -0400, Ronald S. Bultje wrote: >> > --- >> > libavcodec/dirac_arith.c | 17 +++-- >

Re: [FFmpeg-devel] [PATCH] doc/examples/extract_mvs: switch to codecpar

2017-03-28 Thread Matthieu Bouron
On Tue, Mar 28, 2017 at 12:53 PM, Matthieu Bouron wrote: > > > On Tue, Mar 28, 2017 at 1:48 PM, Matthieu Bouron < > matthieu.bou...@gmail.com> wrote: > >> --- >> doc/examples/extract_mvs.c | 33 ++--- >> 1 file changed, 22 insertions(+), 11 deletions(-) >> >> diff --g

Re: [FFmpeg-devel] [PATCH] doc/examples/remuxing: switch to codecpar

2017-03-28 Thread Matthieu Bouron
On Tue, Mar 28, 2017 at 12:32 PM, Matthieu Bouron wrote: > Also limits remuxing to audio, video and subtitle streams. > --- > doc/examples/remuxing.c | 48 ++ > +++--- > 1 file changed, 41 insertions(+), 7 deletions(-) > > diff --git a/doc/examples/remuxin

Re: [FFmpeg-devel] [PATCH] doc/examples/remuxing: switch to codecpar

2017-03-28 Thread Matthieu Bouron
On Tue, Mar 28, 2017 at 4:21 PM, Moritz Barsnick wrote: > On Tue, Mar 28, 2017 at 12:32:19 +0200, Matthieu Bouron wrote: > > -fprintf(stderr, "Failed to copy context from input to > output stream codec context\n"); > > +fprintf(stderr, "Failed to copy copy codec parameters

Re: [FFmpeg-devel] [PATCH] doc/examples/remuxing: switch to codecpar

2017-03-28 Thread Moritz Barsnick
On Tue, Mar 28, 2017 at 12:32:19 +0200, Matthieu Bouron wrote: > -fprintf(stderr, "Failed to copy context from input to output > stream codec context\n"); > +fprintf(stderr, "Failed to copy copy codec parameters\n"); "copy copy"? Moritz ___

Re: [FFmpeg-devel] [PATCH v2 1/2] avfilter/interlace: change lowpass_line function

2017-03-28 Thread Kieran Kunhya
On Tue, 28 Mar 2017 at 14:17 Thomas Mundt wrote: > >>Thomas Mundt schrieb am Di, 21.3.2017: > >>>Thomas Mundt schrieb am Mo, > 13.3.2017: > >> Hi, > >> > >> this patch set > >> 1) changes the lowpass_line function prototype in vf_interlace and > vf_tinterlace as suggested by James Almer. > >> 2

Re: [FFmpeg-devel] [PATCH v2 1/2] avfilter/interlace: change lowpass_line function

2017-03-28 Thread Thomas Mundt
>>Thomas Mundt schrieb am Di, 21.3.2017: >>>Thomas Mundt schrieb am Mo, 13.3.2017: >> Hi, >> >> this patch set >> 1) changes the lowpass_line function prototype in vf_interlace and >> vf_tinterlace as suggested by James Almer. >> 2) adds a complex (-1 2 6 2 -1) vertical low-pass filter to vf_in

Re: [FFmpeg-devel] [PATCH] avcodec/dnxhdenc: DNxHR 444 and HQX support

2017-03-28 Thread Paul B Mahol
On 3/28/17, Mark Reid wrote: > On Mon, Mar 27, 2017 at 3:14 PM, Paul B Mahol wrote: > >> On 3/27/17, Paul B Mahol wrote: >> > On 3/27/17, Mark Reid wrote: >> >> On Mon, Mar 27, 2017 at 11:08 AM, Paul B Mahol >> wrote: >> >> >> >>> Signed-off-by: Paul B Mahol >> >>> --- >> >>> libavcodec/dnxh

[FFmpeg-devel] Match source video timestamp

2017-03-28 Thread Eran Kornblau
Hi all, I'm trying to transcode some video file (MP4/h264 baseline vfr/AAC) with ffmpeg, and I would like to have the frame timestamps in the transcoded file perfectly match the source video. This is in order to have them both play together adaptively without issues (from my experience, DASH is

Re: [FFmpeg-devel] Refund request for travelling costs to Chemnitz

2017-03-28 Thread Thomas Volkert
On 13.03.2017 11:04, Carl Eugen Hoyos wrote: > Hi! > > Alexander Strasser, Thilo Borgmann, Thomas Volkert and myself > represented FFmpeg at the Chemnitzer Linuxtage 2017. We showed > different filters in action on video screens, talked to guests and > other projects and as usual answered many use

Re: [FFmpeg-devel] [PATCH] doc/examples/remuxing: switch to codecpar

2017-03-28 Thread wm4
On Tue, 28 Mar 2017 13:09:54 +0200 Matthieu Bouron wrote: > On Tue, Mar 28, 2017 at 12:48 PM, wm4 wrote: > > > On Tue, 28 Mar 2017 12:32:19 +0200 > > Matthieu Bouron wrote: > > > > > Also limits remuxing to audio, video and subtitle streams. > > > --- > > > doc/examples/remuxing.c | 48

Re: [FFmpeg-devel] [PATCH] doc/examples/remuxing: switch to codecpar

2017-03-28 Thread Matthieu Bouron
On Tue, Mar 28, 2017 at 12:48 PM, wm4 wrote: > On Tue, 28 Mar 2017 12:32:19 +0200 > Matthieu Bouron wrote: > > > Also limits remuxing to audio, video and subtitle streams. > > --- > > doc/examples/remuxing.c | 48 ++ > +++--- > > 1 file changed, 41 insert

Re: [FFmpeg-devel] [PATCH] doc/examples/extract_mvs: switch to codecpar

2017-03-28 Thread Matthieu Bouron
On Tue, Mar 28, 2017 at 1:48 PM, Matthieu Bouron wrote: > --- > doc/examples/extract_mvs.c | 33 ++--- > 1 file changed, 22 insertions(+), 11 deletions(-) > > diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c > index 975189c77d..09c3d32389 100644 >

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Clément Bœsch
On Tue, Mar 28, 2017 at 12:57:52PM +0200, Matthias Hunstock wrote: > Am 28.03.2017 um 10:19 schrieb Clément Bœsch: > > But you're right on all your comments so I just adjusted the function. See > > patch attached. > > Did you also update the deprecation notice in [PATCH 02/10]? > I didn't rename

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Matthias Hunstock
Am 28.03.2017 um 10:19 schrieb Clément Bœsch: > But you're right on all your comments so I just adjusted the function. See > patch attached. Did you also update the deprecation notice in [PATCH 02/10]? Matthias ___ ffmpeg-devel mailing list ffmpeg-devel

Re: [FFmpeg-devel] [PATCH] doc/examples/remuxing: switch to codecpar

2017-03-28 Thread wm4
On Tue, 28 Mar 2017 12:32:19 +0200 Matthieu Bouron wrote: > Also limits remuxing to audio, video and subtitle streams. > --- > doc/examples/remuxing.c | 48 +--- > 1 file changed, 41 insertions(+), 7 deletions(-) > > diff --git a/doc/examples/remuxing

[FFmpeg-devel] [PATCH] doc/examples/extract_mvs: switch to codecpar

2017-03-28 Thread Matthieu Bouron
--- doc/examples/extract_mvs.c | 33 ++--- 1 file changed, 22 insertions(+), 11 deletions(-) diff --git a/doc/examples/extract_mvs.c b/doc/examples/extract_mvs.c index 975189c77d..09c3d32389 100644 --- a/doc/examples/extract_mvs.c +++ b/doc/examples/extract_mvs.c @@ -6

[FFmpeg-devel] [PATCH] doc/examples/remuxing: switch to codecpar

2017-03-28 Thread Matthieu Bouron
Also limits remuxing to audio, video and subtitle streams. --- doc/examples/remuxing.c | 48 +--- 1 file changed, 41 insertions(+), 7 deletions(-) diff --git a/doc/examples/remuxing.c b/doc/examples/remuxing.c index 65437d9abd..8615c73842 100644 --- a/d

Re: [FFmpeg-devel] [PATCH v2] avformat/flvdec: check FLVHeader PreviousTagSize0

2017-03-28 Thread Steven Liu
2017-03-26 18:39 GMT+08:00 Bodecs Bela : > > > 2017.03.25. 14:15 keltezéssel, Steven Liu írta: > >> refer to SPEC: >> Annex E. The FLV File Format said: >> E.3 TheFLVFileBody have a table: >> FieldTypeComment >> PreviousTagSize0 UI32Always 0 >> >> Signed-off-by: Steven Liu >>

Re: [FFmpeg-devel] [PATCH v9] avformat/dashdec: add dash demuxer base version

2017-03-28 Thread Steven Liu
2017-03-27 8:39 GMT+08:00 Rodger Combs : > > > On Mar 26, 2017, at 19:10, Steven Liu wrote: > > > > v2 fixed: > > 1. from autodetect to disabled > > 2. from camelCase code style to ffmpeg code style > > 3. from RepType to AVMediaType > > 4. fix variable typo > > 5. change time value from uint32_t

Re: [FFmpeg-devel] [PATCH 01/10] lavu: add av_fourcc_make_string() and av_4cc2str()

2017-03-28 Thread Clément Bœsch
On Tue, Mar 28, 2017 at 12:47:39AM +0200, Alexander Strasser wrote: [...] > > +#define AV_FOURCC_MAX_STRING_SIZE 32 > > Should be fine, though I don't know how you came up with this max size in > particular. > IIRC the actual maximum is 21 characters, I just took the closest superior power of t